[lwip-devel] [patch #7855] Provide alternative timeout implementation

[chrysn]

Follow-up Comment #3, patch #7855 (project lwip):

@comparison: `endtime <= now` does not catch the case when now is before the
timer wrap and endtime is after. you'd have to do at least `(endtime >
last_time_i_checked && endtime <= now) || (last_time_i_checked > now &&
endtime <= now)` -- or map times back to be deltas again internally, so that a
subtraction operation will take care of the wraps.

irrespective of that discussion, i support your approach of having a the
timeout functions exchangable, and do welcome an alternative implementation to
provide internal benchmarks. the point of my original comment here was mainly
to state that delta appraoch is not fundamentally broken, and to alert people
who might carelessly want to just drop the delta approach for an absolute
implementation (which wouldn't be you, but others who experience problems with
timeouts and wind up here).

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?7855>

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