[lwip-devel] [bug #51593] Fix swapped src/dst params with ip4_route_src

[David van Moolenbroek]

Follow-up Comment #5, bug #51593 (project lwip):

Just my 2c there: I think that as a general software development principle, it
is a good idea to push the bugfix immediately and separately from such a
larger otherwise-nonfunctional API change in just about all cases. Not only
may the API change take a while longer and thus needlessly leave the bug
unsolved in the meantime, but also, consider for example what happens when
later on, someone disagrees with the new order of the parameters and rolls
back the API change commit they thought was purely nonfunctional, thereby
reintroducing the bugs as well. I am guilty of violating that principle myself
on many occasions, but I think it's a good idea nonetheless.. :)

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?51593>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/