Wireshark-bugs: [Wireshark-bugs] [Bug 8161] USBHUB: Add useful details to INFO column
Date: Thu, 03 Jan 2013 16:47:54 +0000

changed bug 8161

What Removed Added
CC   mmann78@netscape.net

Comment # 4 on bug 8161 from
(In reply to comment #3)
> Personally I find it
> more difficult to follow code when function calls are nested as you suggest.
> Surely the compiler is smart enough to optimize away any inefficiency that
> having a separate variable might cause...?

I believe some compilers are smart enough to complain about the lack of
optimization.  I don't have a strong opinion either way, I just thought it was
an easier fix than moving "feature_name" to the top of the function since it
would no longer be "locally nested" by the check_col() check.


You are receiving this mail because:
  • You are watching all bug changes.