<div dir="ltr"><br><br><div class="gmail_quote">On Wed, Mar 4, 2015 at 3:06 PM John Clements <<a href="mailto:johnbclements@gmail.com">johnbclements@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>I see that the documentation suggests that (entity-charset) is supposed to return a symbol. However, it nearly always returns a string. In particular, it appears to me that it returns a symbol only when it returns its default, 'us-ascii.<br><br></div>I feel compelled to repair this, but there are two ways to fix it:<br></div><div>1) make it match the docs and always return a symbol, or<br></div><div>2) change the docs and the default to return a string.<br><br></div><div>It looks to me like #2 will break (less) code, though it's certainly possible that people depend on the default value's being a string.</div></div></blockquote><div><br></div><div>It seems like option #3, document the current behavior, will break the least code, and that we should do that.</div><div><br></div><div>Sam </div></div></div>