[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [bug-gettext] JavaScript support
From: |
Andreas Stricker |
Subject: |
Re: [bug-gettext] JavaScript support |
Date: |
Tue, 16 Apr 2013 23:50:35 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 |
Daiki Ueno wrote:
> The attached is a revised patch based on yours found on github:
> https://github.com/AndyStricker/gettext-javascript
Thanks for sanitizing my gettext-javascript branch into a single patch.
> * remove support for '\a' and '\U', which are not supported by
> JavaScript
Good catch.
> * fix multiline comment handling (/* ... */)
I've to look into that, strange we didn't step onto this in our
Javascript codebase.
> * make newline acceptable in regular expression literals
Again good catch. I'll extend the test suit with this one and
the multiline comments above.
> * make string concatenation code in x_javascript_lex more readable
> * remove INTERPRET_ANSIC and INTERPRET_UNICODE option from
> phase7_getuc, since raw strings are not supported by JavaScript
Too much copy & paste by me :-/
> * treat 'else' as well as 'return' for div vs. regex check
You've already added this to the test suite I see.
> * add Unicode escape test
Yes, this must not be missing from the test suite.
> * update document and FILES
Perfect.
> By the way, which is the latest development branch: javascript or
> js-rebased?
Well they should be identical: The rebase was an attempt to make it
easier to merge during the long time of silence.
> Also, I still don't get the idea of this change:
> https://github.com/AndyStricker/gettext-javascript/commit/44008a5ba866ba184f667853672846f50e27f844
Hmm, I'm curious too, the commit didn't highlight _why_ it needs
to check for equality. Currently I'm to tired to grasp this...
Thanks for your intense review on this patch. What's still left
to be done before it can be integrated? I'll run it over our code
base to check if there any problems and bugs left.
Regards, Andy
signature.asc
Description: OpenPGP digital signature