On 06/05/2016 12:56, Dimitry Sibiryakov wrote: > 06.05.2016 17:44, Adriano dos Santos Fernandes wrote: >> It should throw an error if the string is an absolute path. > Ok, but what it must to do if appended path is relative? For example, what > should be > result of "/usr/bin/firebird" += "plugins/" ? > "/usr/bin/firebirdplugins/" or "/usr/bin/firebird/plugins/" ? > Or do you insist on explicit call of ensureSeparator() at each step? > > I, actually, like the idea of prohibiting '+' and '+=' operators and > enforcing of using > .appendPath() and .appendString() methods for PathName.
We already have PathUtils::concatPath doing this correctly (putting an separator between the parts). I just think it should throw error when the second path is absolute, as there is no sense in doing this. Adriano ------------------------------------------------------------------------------ Find and fix application performance issues faster with Applications Manager Applications Manager provides deep performance insights into multiple tiers of your business applications. It resolves application problems quickly and reduces your MTTR. Get your free trial! https://ad.doubleclick.net/ddm/clk/302982198;130105516;z Firebird-Devel mailing list, web interface at https://lists.sourceforge.net/lists/listinfo/firebird-devel
