[Xcb] Proposed patch to support clang compilation of util-image

[Josh Triplett]

On Mon, Oct 17, 2016 at 07:44:09AM +0000, Bart Massey wrote:
> Yeah, assert() should be marked noreturn (directly or indirectly) in
> general or bad things happen. Those assert()s really want to be assert()s
> or calls to abort() or something: there is nothing good that can happen
> after returning from those places.

I just checked, and clang definitely does support the noreturn
attribute.  And the system headers typically define assert with the
noreturn attribute.

Did the build occur with NDEBUG defined or similar, to no-op out the
assert?