Code review comment for lp:~zorba-coders/zorba/bug955040

Revision history for this message
Cezar Andrei (cezar-andrei) wrote :

1. ZORBA_WITH_FOP means I really want FOP. Ie if there is anything missing it's an error.
So if ZORBA_WITH_FOP and ZORBA_SUPPRESS_JAVA are both true should be an error.

2. Why did you replace 1 IF with AND condition with 8 nested IF's ?

review: Needs Fixing

« Back to merge proposal