Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1034582-flexlexerh into lp:zorba

2012-08-09 Thread Paul J. Lucas
Review: Approve

Yep, it works on Mountain Lion using the stock flex.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-1034582-flexlexerh/+merge/119075
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1034582-flexlexerh into lp:zorba

2012-08-09 Thread Paul J. Lucas
Why do you copy FlexLexer.h to the top-level src directory?  Why isn't it 
copied to build/src/compiler/parser to mirror the pre-generated location?
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-1034582-flexlexerh/+merge/119075
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1034582-flexlexerh into lp:zorba

2012-08-09 Thread Chris Hillery
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-1034582-flexlexerh/+merge/119075
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp