preprocessor build patch
The c preprocessor kicks out occasional warnings as it moves through the Objective-J code. These warnings were being sucked into the output by the ant build system. This patch separates the warnings into a different file where they can be analyzed (or ignored).
The warning I was getting was the preprocessor complaining about a particularly nasty regular expression not being quoted properly. It's valid javascript it just doesn't parse as c.
-
Brian TaylorFYI:
uname Linux buster 2.6.24-19-generic #1 Finish setting up lighthouse. SMP Wed Aug 20 17:53:40 UTC 2008 x86_64 GNU/Linux
gcc -v Configured with: ../src/configure -v --enable-languages=c,c++,fortran,objc,obj-c++,treelang --prefix=/usr --enable-shared --with-system-zlib --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --enable-nls --with-gxx-include-dir=/usr/include/c++/4.2 --program-suffix=-4.2 --enable-clocale=gnu --enable-libstdcxx-debug --enable-objc-gc --enable-mpfr --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu Thread model: posix gcc version 4.2.4 (Ubuntu 4.2.4-1ubuntu3)
-
Francisco Tolmasky- Assigned user set to Francisco Tolmasky
I'll go ahead and throw this in, by any chance could you show me that Regex, I'm curious to see what GCC says
-
Brian Taylor
It's the Regex on line 87 of Objective-J/preprocess.js.
GCC says:
warning: missing terminating " character
This warning output gets sucked into objj.build/Release/objj.js when Objecive-J.concat is preprocessed. Note the misspelling of "Objective"--I'm submitting a patch to correct that as well.
On Fri, Nov 7, 2008 at 2:00 PM, Lighthouse support@lighthouseapp.comwrote:
-
Brian Taylor
Corrects misspelling of Objective-J.concat in build.
-
Francisco Tolmasky
Good catches, going in now
-
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.