The Guidelines should be in good shape already. I made sure the correct function calls were there already, and added the verbiage to the add_theme_support() Codex page regarding Themes not making direct calls for those arguments.<div>
<br></div><div>Chip<br><br><div class="gmail_quote">On Sun, Sep 19, 2010 at 7:22 PM, Edward Caissie <span dir="ltr">&lt;<a href="mailto:edward.caissie@gmail.com">edward.caissie@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Thanks for the &quot;heads up&quot;, Nacin;<br><br>We&#39;ll look into referencing this on the Theme Review page(s), too.<br><br><br>Cais.<br><br><div class="gmail_quote"><div><div></div><div class="h5">On Sun, Sep 19, 2010 at 8:16 PM, Andrew Nacin <span dir="ltr">&lt;<a href="mailto:wp@andrewnacin.com" target="_blank">wp@andrewnacin.com</a>&gt;</span> wrote:<br>


</div></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex"><div><div></div><div class="h5">A heads up, I&#39;ve removed references to read-only parameters for add_theme_support() from <a href="http://codex.wordpress.org/Function_Reference/add_theme_support" target="_blank">http://codex.wordpress.org/Function_Reference/add_theme_support</a>.<div>




<br></div><div>Any theme enabling &#39;editor-style&#39;, &#39;menus&#39;, &#39;custom-header&#39;, &#39;custom-background&#39;, or &#39;widgets&#39; via add_theme_support() is doing it wrong. These are internal registrations so we can transparently use current_theme_support(). In fact WordPress prevents these in particular from being removed via remove_theme_support(), that way there are no imbalances.</div>




<div><br></div><font color="#888888"><div>Nacin</div>
</font><br></div></div>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>