#127 ✓resolved
Brian Taylor

preprocessor build patch

Reported by Brian Taylor | November 1st, 2008 @ 05:44 AM

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.

Comments and changes to this ticket

  • Brian Taylor

    Brian Taylor November 1st, 2008 @ 07:12 AM

    FYI:

    uname Linux buster 2.6.24-19-generic #1 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

    Francisco Tolmasky November 7th, 2008 @ 09:00 AM

    • 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

    Brian Taylor November 7th, 2008 @ 01:35 PM

    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

    Brian Taylor November 7th, 2008 @ 01:59 PM

    Corrects misspelling of Objective-J.concat in build.

  • Francisco Tolmasky
  • Francisco Tolmasky

    Francisco Tolmasky November 8th, 2008 @ 11:01 AM

    • State changed from “new” to “resolved”

    k, its in

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.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

The Cappuccino Web Framework, including AppKit, Foundation, and Objective-J.

Shared Ticket Bins

People watching this ticket

Tags

Pages