15.03.2016 14:05, Dimitry Sibiryakov wrote: > In attachment you can find patch for CORE-3172 and another one that adds > minimal support for unicode database names into engine.
You moved class WideCharBuffer into os_utils, as suggested, it is ok. But why you not included it into namespace os_utils ??? Also, i prefer to see is at separate commit. > Together they make attached testcase work. > These patches target only particular issue and are minimal on purpose. Does POSIX requires any changes for this issue ? > Question: should I commit them at once or separately? I see two questions in statement above. Vlad ------------------------------------------------------------------------------ Transform Data into Opportunity. Accelerate data analysis in your applications with Intel Data Analytics Acceleration Library. Click to learn more. http://pubads.g.doubleclick.net/gampad/clk?id=278785231&iu=/4140 Firebird-Devel mailing list, web interface at https://lists.sourceforge.net/lists/listinfo/firebird-devel
