Re: [Firebird-devel] LocalStatus - check or throw

2015-04-06 Thread Dimitry Sibiryakov
06.04.2015 11:59, Alex Peshkoff wrote: > CheckStatusWrapper is used in sources > hundreds times. Yep, you are right. My fault. -- WBR, SD. -- BPM Camp - Free Virtual Workshop May 6th at 10am PDT/1PM EDT Develop yo

Re: [Firebird-devel] LocalStatus - check or throw

2015-04-06 Thread Alex Peshkoff
On 04/06/15 12:47, Dimitry Sibiryakov wrote: > 06.04.2015 11:31, Alex Peshkoff wrote: >> I've thought that you suggest to use ThrowStatusWrapper instead >> CheckStatusWrapper:) > No, I wanted to know a reason (if any) for using CheckStatusWrapper > there. Because it > is the only place where i

Re: [Firebird-devel] LocalStatus - check or throw

2015-04-06 Thread Dimitry Sibiryakov
06.04.2015 11:31, Alex Peshkoff wrote: > I've thought that you suggest to use ThrowStatusWrapper instead > CheckStatusWrapper:) No, I wanted to know a reason (if any) for using CheckStatusWrapper there. Because it is the only place where it is used. -- WBR, SD. -

Re: [Firebird-devel] LocalStatus - check or throw

2015-04-06 Thread Alex Peshkoff
On 04/06/15 11:35, Dimitry Sibiryakov wrote: > 06.04.2015 10:18, Alex Peshkoff wrote: >> You are absolutely right > "Right?" I've thought that you suggest to use ThrowStatusWrapper instead CheckStatusWrapper :) > I only try to find out use cases for the wrappers: when I must use > Check

Re: [Firebird-devel] LocalStatus - check or throw

2015-04-06 Thread Dimitry Sibiryakov
06.04.2015 10:18, Alex Peshkoff wrote: > You are absolutely right "Right?" I only try to find out use cases for the wrappers: when I must use CheckStatusWrapper and when ThrowStatusWrapper. I thought that I understand their use until have found the code. -- WBR, SD. --

Re: [Firebird-devel] LocalStatus - check or throw

2015-04-06 Thread Dimitry Sibiryakov
06.04.2015 10:18, Alex Peshkoff wrote: >> 1) In pluginmanager.cpp::findDefConfig() the only thing that is done with >> status vector is >> call of check() that throw. Why ThrowStatusWrapper isn't used there? > > You are absolutely right - sometimes we will always (well, may be not > always but def

Re: [Firebird-devel] LocalStatus - check or throw

2015-04-06 Thread Alex Peshkoff
On 04/04/15 16:20, Dimitry Sibiryakov wrote: > Hello, All. > > I have two question about latest changes in interfaces: > > 1) In pluginmanager.cpp::findDefConfig() the only thing that is done with > status vector is > call of check() that throw. Why ThrowStatusWrapper isn't used there? Yo

[Firebird-devel] LocalStatus - check or throw

2015-04-04 Thread Dimitry Sibiryakov
Hello, All. I have two question about latest changes in interfaces: 1) In pluginmanager.cpp::findDefConfig() the only thing that is done with status vector is call of check() that throw. Why ThrowStatusWrapper isn't used there? 2) Why wrapper is needed at all instead of declaring LocalSta