zorba-coders team mailing list archive

[Chris Hillery <chillery+launchpad@xxxxxxxxx>]

Review: Needs Fixing

You can't remove path_util.cpp - it also handles the --lib-path and --uri-path command-line options, not just the environment variables. So you can get rid of getPathFromEnvironment(), and the parts of setPathsOnContext() which reference lEnvStr, but the rest has to stay, including the call to setPathsOnContext() in zorbacmd.cpp.

(Unrelated to your change, but it does appear that concatenatePaths() is unused code, so you could remove that too.)
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1058231/+merge/129531
Your team Zorba Coders is subscribed to branch lp:zorba.