<br><br><div class="gmail_quote">On Fri, Mar 5, 2010 at 10:28 AM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">



  

<div><div class="im">
On 3/5/10 4:13 PM, Myles Watson wrote:<br>
<blockquote type="cite">
  <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;">I
think we should get rid of that warning, since we conditionally call
functions based on debugging and various config variables.  </blockquote>
  </div>
</blockquote></div>
I think we should instead just drop CONFIG_WARNINGS_ARE_ERRORS for Qemu
rather than dropping warnings.</div></blockquote><div>I think that setting for Qemu has reduced the number of warnings in our code by a lot.  It's easier for the original developer to deal with warnings when the code is written.<br>
 <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div>Or maybe only add -Wno-unused-function
if <br></div></blockquote><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div text="#000000" bgcolor="#ffffff">CONFIG_WARNINGS_ARE_ERRORS is active, at least.  Or maybe add an
Option "Warnings good for Developers"?</div></blockquote><div>I can see your point.  I'd rather not have to special case warnings either.  What would you suggest for the two emulator functions in yabel that are unused right now?  Should we comment them out until they are used?<br>
 <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im"><div class="im">
<blockquote type="cite">
  <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;">Is
there a case where it helps enough to justify all the warnings?<br>
  </blockquote>
  </div>
</blockquote>
</div>
Yes, they indicate that there is dead code. This is, not only but
especially useful to recognize if / how code should / could be
restructured. <br></div></blockquote><div>I worry that having too many warnings makes it so that "important" ones (ones that cause bugs) get missed. <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">
<blockquote type="cite">
  <div class="gmail_quote">
  <div> Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com" target="_blank">mylesgw@gmail.com</a>><br>
  </div>
  </div>
</blockquote></div></blockquote><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div text="#000000" bgcolor="#ffffff">
With the -Wno-unused-functions taken care of:<br></div></blockquote><div> <br>I wasn't sure how you wanted it taken care of.  I left it for another patch.<br></div><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div text="#000000" bgcolor="#ffffff">Acked-by: Stefan Reinauer <a href="mailto:stepan@coresystems.de" target="_blank"><stepan@coresystems.de></a><br></div></blockquote><div><br>Rev 5191.<br><br>Thanks,<br>Myles <br>
</div></div>