<div dir="ltr">Urghh.... really? The existing behavior is clearly broken, and this library is--to the best of my knowledge--used by a relatively small number of people. Francisco, as the original author of this code, do you have an opinion?<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 4, 2015 at 12:13 PM, Sam Tobin-Hochstadt <span dir="ltr"><<a href="mailto:samth@cs.indiana.edu" target="_blank">samth@cs.indiana.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><br><div class="gmail_quote"><span class="">On Wed, Mar 4, 2015 at 3:06 PM John Clements <<a href="mailto:johnbclements@gmail.com" target="_blank">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></span><div>It seems like option #3, document the current behavior, will break the least code, and that we should do that.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Sam </div></font></span></div></div>
</blockquote></div><br></div>