[erlang-patches] Fixes for unicode handling in re module
Rory Byrne
rory@REDACTED
Sun Jan 17 21:14:25 CET 2010
On Sun, Jan 17, 2010 at 05:07:57PM +0100, Björn Gustavsson wrote:
> On Sat, Jan 16, 2010 at 6:46 PM, Rory Byrne <rory@REDACTED> wrote:
> > I've added fixes for three unicode-related bugs in the re module:
> >
> > git fetch git://github.com/jinsky/otp.git stdlib_re_unicode_fixes
>
> Thanks!
>
> I have removed some trailing whitespace (using
> "git rebase --whitespace=fix") and included the result in 'pu'.
Good tip, I fixed my branch. Sorry about that.
> I also thinks that the commit that introduces a failing
> test case should be combined with the commit that
> corrects the bug, in order to keep the history bisectable:
>
> http://www.kernel.org/pub/software/scm/git/docs/git-bisect-lk2009.html
>
> If you agree that is a good change, you can either
> combine the commits yourself or I could do it for you.
> If you have a strong reason for keeping the commits
> separate, I would like to hear about it.
Sounds good, don't know what I was thinking.
I used "git rebase -i" to squash the commits and then did a
"git push --force ..." to update the remote branch.
Hope this is what you had in mind.
Thanks,
Rory
More information about the erlang-patches
mailing list