Details
- Reviewers
karels
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 48247 Build 45133: arc lint + arc unit
Event Timeline
My main question is whether the DEBUG definitions should just be deleted, or whether it is worth leaving bread crumbs for where it used to be defined.
lib/libc/net/res_config.h | ||
---|---|---|
3 | Shouldn't this just be deleted rather than doing #undef? If the user defines DEBUG then includes headers, or defines DEBUG on the command line, I think it should be honored, | |
lib/libc/resolv/res_mkquery.c | ||
87 | I wonder whether the previous two lines should just be deleted (similarly elsewhere). |
lib/libc/net/res_config.h | ||
---|---|---|
3 | I'm not sure the two cases are analogous. If I understand correctly, there is no reason to enable SUNSECURITY at all. However, DEBUG might reasonably enabled for debugging, etc. It makes the debug option to the resolver actually do something. btw, why did you remove the comment on SUNSECURITY? I wouldn't change that line at all. | |
lib/libc/resolv/res_mkquery.c | ||
87 | True, but the comment no longer makes sense. |