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