<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div bgcolor="#ffffff" text="#000000"><div><br>
<blockquote type="cite">BIOS_NEVER -> BIOS_SPEW+1<br>
</blockquote></div>
The whole intent was to make clear to the reader that this is something
that's never seen. ;-)<br>
That doesn't become clear from BIOS_SPEW+1 in my opinion.</div></blockquote></div><div>NEVER was only used once or twice in the code.  If it's something that will be used more often, then I can see your point.  Why don't we just comment out messages that will never be seen?<br>
</div></div></blockquote></div><br>I just looked again, and it was only used once.  It makes sense where it was used, but I don't think it makes sense to use it other places, and I think naming it with the same prefix and including it in the same list makes that likely in the future.<br>
<br>I think a comment explaining BIOS_SPEW+1 would work just as well, or this:<br>BIOS_NEVER -> CONFIG_MAXIMUM_CONSOLE_LOGLEVEL+1<br><br>Thanks,<br>Myles<br>