[tor-bugs] #13577 [Tor]: Fix clang compilation warnings in tor, trunnel
Tor Bug Tracker & Wiki
blackhole at torproject.org
Thu Oct 30 09:55:48 UTC 2014
#13577: Fix clang compilation warnings in tor, trunnel
------------------------+------------------------------
Reporter: teor | Owner:
Type: defect | Status: needs_review
Priority: minor | Milestone:
Component: Tor | Version: Tor: unspecified
Resolution: | Keywords: clang
Actual Points: | Parent ID:
Points: |
------------------------+------------------------------
Comment (by teor):
Yes, !! is pretty ugly, isn't it?
I think -Wno-parenthesis-equality is the best option, as it's obviously
not adding that much value if it's causing these sorts of false positives.
But I'm concerned that it is actually adding value in some cases (i.e. =
vs == in conditionals)
I also have doubts about moving the !! into the macro, because !! would
have to sit outside the macro's outer brackets to be effective. And that
causes its own issues.
We could use inline functions, but that might break some of the
consistency / encapsulation / optimisation. I'll look into it.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/13577#comment:3>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list