Well, I'm going to package up a release tomorrow afternoon, so you guys better let me know what you want included. :)<br><br>Julian<br><br><div class="gmail_quote">On Fri, May 21, 2010 at 1:27 PM, Paolo Bonzini <span dir="ltr"><<a href="mailto:bonzini@gnu.org">bonzini@gnu.org</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;">On Fri, May 21, 2010 at 13:35, Philippe Marschall<br>
<div class="im"><<a href="mailto:philippe.marschall@gmail.com">philippe.marschall@gmail.com</a>> wrote:<br>
> 2010/5/21 Julian Fitzell <<a href="mailto:jfitzell@gmail.com">jfitzell@gmail.com</a>>:<br>
>> On Fri, May 21, 2010 at 8:46 AM, Philippe Marschall<br>
>> <<a href="mailto:philippe.marschall@gmail.com">philippe.marschall@gmail.com</a>> wrote:<br>
>>><br>
>>> 2010/5/21 Julian Fitzell <<a href="mailto:jfitzell@gmail.com">jfitzell@gmail.com</a>>:<br>
>>> > So when you say hacked in, you mean I should try not to include these<br>
>>> > versions this weekend? :)<br>
>>><br>
>>> No, by all means include it so that it gets tested. What I mean is the<br>
>>> code is probably not going to stay in the state it is currently in.<br>
>>> Which doesn't matter so much for Pharo but matters for everybody else<br>
>>> who wants to build an optimized implementation for his platform.<br>
>>><br>
</div><div class="im">>>> The end result is methods on platform for escaping xml and urls with<br>
>>> the current code the default implementation on GRPlatform.<br>
<br>
</div>I think "^true" will be faster for the default platform, especially for<br>
#includesUnsafeUrlCharacter: If you interpret that as the method<br>
having false positives, but not false negatives, you can even<br>
keep some testcases.<br>
<br>
With this change I have no reason to complain about this optimization.<br>
<font color="#888888"><br>
Paolo<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
seaside-dev mailing list<br>
<a href="mailto:seaside-dev@lists.squeakfoundation.org">seaside-dev@lists.squeakfoundation.org</a><br>
<a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" target="_blank">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br>
</div></div></blockquote></div><br>