<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Feb 8, 2016 at 9:46 AM Nico Huber <<a href="mailto:nico.h@gmx.de">nico.h@gmx.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 08.02.2016 12:10, Patrick Georgi via coreboot wrote:<br>
> 2016-02-04 10:35 GMT+01:00 Patrick Georgi <<a href="mailto:pgeorgi@google.com" target="_blank">pgeorgi@google.com</a>>:<br>
>> during the review of some commits that are in the process of being<br>
>> upstreamed from Chrome OS, people noticed that chipset drivers like to<br>
>> define their own TRUE/FALSE defines (sometimes prefixed to), and I<br>
>> have seen a bunch of #define BIT{0-31} ..., too, because that seems to<br>
>> be the house rules in some firmware communities.<br>
> Seems like for the BIT defines, all variants are popular. Any<br>
> objection with moving them to src/include instead of having various<br>
> copies across the tree?<br>
How about something like src/include/please_dont_use.h ?<br><br></blockquote><div><br></div><div><br></div><div>I don't know about that but I'd have no objection to a comment in the file itself that these are vile constructs. </div><div><br></div><div>ron </div></div></div>