[haiku-bugs] Re: [Haiku] #12825: [Patch] net_buffer.cpp: Fix clang warning

  • From: "axeld" <trac@xxxxxxxxxxxx>
  • Date: Mon, 11 Jul 2016 20:30:06 -0000

#12825: [Patch] net_buffer.cpp: Fix clang warning
----------------------------------------+----------------------------
   Reporter:  mt                        |      Owner:  axeld
       Type:  bug                       |     Status:  new
   Priority:  normal                    |  Milestone:  Unscheduled
  Component:  Network & Internet/Stack  |    Version:  R1/Development
 Resolution:                            |   Keywords:
 Blocked By:                            |   Blocking:
Has a Patch:  1                         |   Platform:  All
----------------------------------------+----------------------------

Comment (by axeld):

 I'd prefer to use a `while (true)` loop here; especially since leaving it
 like this produces a warning, and is also slightly confusing for those who
 expect loop conditions make sense, and don't just hint at the intended
 usage :-)

 In any case, the suggested patch is not correct, and would break the code,
 nice catch, Adrien!

--
Ticket URL: <https://dev.haiku-os.org/ticket/12825#comment:4>
Haiku <https://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: