The issue with keeping them in the main compat header is that the macro
definitions leak into anything that includes zstd.h, which might include
native FreeBSD code.
Details
Details
Diff Detail
Diff Detail
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 15010 Build 15117: arc lint + arc unit
Event Timeline
Comment Actions
this looks fine, apart maybe from the copyright stuff...
But honestly, those few lines are likely not deserving of copyright protection...
sys/contrib/zstd/lib/freebsd/stdlib.h | ||
---|---|---|
2 | 2015? |
Comment Actions
I didn't really think so either, but it doesn't hurt to be conservative here.
sys/contrib/zstd/lib/freebsd/stdlib.h | ||
---|---|---|
2 | I just copied it from zstd_kfreebsd.h. |
sys/contrib/zstd/lib/freebsd/stdlib.h | ||
---|---|---|
2 | Fair enough. |