Re: [Firebird-devel] IUtl functions

2014-01-26 Thread Adriano dos Santos Fernandes
On 20-01-2014 12:40, Alex Peshkoff wrote: On 01/17/14 19:33, Dmitry Yemanov wrote: 17.01.2014 19:16, Adriano dos Santos Fernandes wrote: I think IUtl::version Being there, please rename it to getVersion(). As usual, I'm strongly against nouns used as routine names. Done. There is a

Re: [Firebird-devel] IUtl functions

2014-01-26 Thread Александр Пешков
On 01/27/14 05:07, Adriano dos Santos Fernandes wrote: On 20-01-2014 12:40, Alex Peshkoff wrote: On 01/17/14 19:33, Dmitry Yemanov wrote: 17.01.2014 19:16, Adriano dos Santos Fernandes wrote: I think IUtl::version Being there, please rename it to getVersion(). As usual, I'm strongly against

Re: [Firebird-devel] IUtl functions

2014-01-20 Thread Alex Peshkoff
On 01/17/14 19:33, Dmitry Yemanov wrote: 17.01.2014 19:16, Adriano dos Santos Fernandes wrote: I think IUtl::version Being there, please rename it to getVersion(). As usual, I'm strongly against nouns used as routine names. Done. Also I think IUtl::edit should not exist. It's a too

[Firebird-devel] IUtl functions

2014-01-17 Thread Adriano dos Santos Fernandes
Alex and All, I think some functions should not be added to new interfaces just to replicate old ones. I think IUtl::version should not call a IVersionCallback. IMO it should receive a buffer and length and write to it or return how many bytes are required for the buffer. Not very handy, but

Re: [Firebird-devel] IUtl functions

2014-01-17 Thread Dmitry Yemanov
17.01.2014 19:16, Adriano dos Santos Fernandes wrote: I think IUtl::version Being there, please rename it to getVersion(). As usual, I'm strongly against nouns used as routine names. Also I think IUtl::edit should not exist. It's a too specialized ISQL feature, and IMO should be there in