[Intel-gfx] [PATCH i-g-t] lib/igt_core: Move write_stderr out of LIBUNWIND ifdef
Arkadiusz Hiler
arkadiusz.hiler at intel.com
Fri Nov 24 11:07:39 UTC 2017
On Fri, Nov 24, 2017 at 10:51:38AM +0000, Chris Wilson wrote:
> Quoting Arkadiusz Hiler (2017-11-24 10:32:07)
> > write_stderr() and __write_stderr() are defined behind ifdef on
> > HAVE_LIBUNWIND, but do no depend on the lib in any way.
>
> Hmm, we keep getting caught out by large ifdefs. Although I guess
> conditionally enabled code itself is less-than-welcome, we should
> probably aim to make whole C files be conditionally compiled, exporting
> no-op interfaces when not enabled. Just a thought.
> -Chris
Sounds about right, and... unwinding the libunwind mess would be a good
way to start.
Any volunteers? I may be able to give it a shot next week...
--
Cheers,
Arek
More information about the Intel-gfx
mailing list