Carl-Daniel, please run 'svn up' in the source directory before committing, as the patch won't apply directly as-is. svn will handle the merge correctly.<br>Thanks!<br><br><div class="gmail_quote">On 15 October 2010 01:47, Sean Nelson <span dir="ltr"><<a href="mailto:audiohacked@gmail.com">audiohacked@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im"> On 9/29/10 5:41 PM, Carl-Daniel Hailfinger wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Replace sizeof("string")-1 with strlen("string")<br>
<br>
We want to avoid calls to strlen at runtime if the string is already<br>
known at compile time.<br>
Turns out that gcc and clang will recognize constant strings and compute<br>
the strlen result already at compile time, so trickery with sizeof only<br>
reduces readability but does not improve the code.<br>
<br>
Signed-off-by: Carl-Daniel Hailfinger<<a href="mailto:c-d.hailfinger.devel.2006@gmx.net" target="_blank">c-d.hailfinger.devel.2006@gmx.net</a>><br>
</blockquote></div>
Acked-by: Sean Nelson <<a href="mailto:audiohacked@gmail.com" target="_blank">audiohacked@gmail.com</a>><div><div></div><div class="h5"><br>
<br>
<br>
_______________________________________________<br>
flashrom mailing list<br>
<a href="mailto:flashrom@flashrom.org" target="_blank">flashrom@flashrom.org</a><br>
<a href="http://www.flashrom.org/mailman/listinfo/flashrom" target="_blank">http://www.flashrom.org/mailman/listinfo/flashrom</a><br>
</div></div></blockquote></div>