Re: [Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Dimitry Sibiryakov
31.05.2016 21:55, Adriano dos Santos Fernandes wrote: > On 31/05/2016 13:58, Dimitry Sibiryakov wrote: >> Yes, you could save time you wasted protecting status quo with simple >> "why not, it >> won't hurt". If you didn't noticed, I just suggested to add a rule to guide, >> not to start >>

Re: [Firebird-devel] Move semantic

2016-05-31 Thread Egor Pugin
> AIX or HPUX with native compilers ... Do these platforms have gcc/clang? If yes why some native old compilers should be taken into account? It's obvious that those compilers almost always will be behind gcc/clang/vc. On 31 May 2016 at 22:53, Adriano dos Santos Fernandes

Re: [Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Adriano dos Santos Fernandes
On 31/05/2016 13:58, Dimitry Sibiryakov wrote: >Yes, you could save time you wasted protecting status quo with simple "why > not, it > won't hurt". If you didn't noticed, I just suggested to add a rule to guide, > not to start > rewriting code. > Your rule breaks the most used class in the

Re: [Firebird-devel] Move semantic

2016-05-31 Thread Adriano dos Santos Fernandes
On 31/05/2016 11:00, Alex Peshkoff wrote: > On 05/31/2016 04:53 PM, Dimitry Sibiryakov wrote: >> 31.05.2016 15:27, Alex Peshkoff wrote: >>> But what about AIX or HPUX with native compilers - probably you need to >>> google carefully re compilers compatibility and tell us how does it look >>> like

[Firebird-devel] Fwd: [Firebird-checkins] [FirebirdSQL/firebird] 7ad99b: Added support for system privileges

2016-05-31 Thread Alex Peshkoff
Hi all! After this commit ODS version became incompatible with FB3. SYSDBA does not have 'god' rights. Log Message: --- Added support for system privileges Sorry for inconvenience - this will be fixed when we decide do we have major or minor ODS change in fb4. Alex.

Re: [Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Dimitry Sibiryakov
31.05.2016 18:41, Alex Peshkoff wrote: > Function is used to merge lists of client and server plugins. I doubt > it's worth optimization. I know what it does. Problem is that it does something more, something that is not obvious from first glance. > Certainly pop > performs constructor, it

Re: [Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Alex Peshkoff
On 05/31/2016 06:55 PM, Dimitry Sibiryakov wrote: > 31.05.2016 17:34, Alex Peshkoff wrote: >> I hardly believe strings can affect performance seriously. >> They are typically not used for objects in deep layers. > Yes, I doubt that these little things can win more than a couple of > percents,

Re: [Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Adriano dos Santos Fernandes
On 31/05/2016 13:24, Dimitry Sibiryakov wrote: > 31.05.2016 18:14, Adriano dos Santos Fernandes wrote: >> We don't need to be aware of non important things, string must be easy >> to use, period. >Then you cannot be sure that none of these places is performance critical. >Speed of code

Re: [Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Alex Peshkoff
On 05/31/2016 07:14 PM, Adriano dos Santos Fernandes wrote: > On 31/05/2016 13:09, Dimitry Sibiryakov wrote: >> 31.05.2016 17:54, Adriano dos Santos Fernandes wrote: >>> String class is general class to be used everywhere except in >>> performance needed code. >>> >>> As a general class, should be

Re: [Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Dimitry Sibiryakov
31.05.2016 18:14, Adriano dos Santos Fernandes wrote: > We don't need to be aware of non important things, string must be easy > to use, period. Then you cannot be sure that none of these places is performance critical. Speed of code writing is unimportant. Only speed of code execution is.

Re: [Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Dimitry Sibiryakov
31.05.2016 17:54, Adriano dos Santos Fernandes wrote: > String class is general class to be used everywhere except in > performance needed code. > > As a general class, should be simple to use with implicit constructors > and conversions. This is good, but are you aware how many places in code

Re: [Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Adriano dos Santos Fernandes
On 31/05/2016 12:22, Dimitry Sibiryakov wrote: >Hello, All. > >During string class refactoring I tried to declare all constructors > explicit because > they are relatively heavy in term of data moving. And it was amazing to see > in how many > places in code compiler is implicitly

[Firebird-devel] Implicitly created temporary objects

2016-05-31 Thread Dimitry Sibiryakov
Hello, All. During string class refactoring I tried to declare all constructors explicit because they are relatively heavy in term of data moving. And it was amazing to see in how many places in code compiler is implicitly creating temporary objects. I'm sure that in some cases it was

Re: [Firebird-devel] Move semantic

2016-05-31 Thread Alex Peshkoff
On 05/31/2016 04:53 PM, Dimitry Sibiryakov wrote: > 31.05.2016 15:27, Alex Peshkoff wrote: >> But what about AIX or HPUX with native compilers - probably you need to >> google carefully re compilers compatibility and tell us how does it look >> like to use this feature on them. I understand it's

Re: [Firebird-devel] Move semantic

2016-05-31 Thread Dimitry Sibiryakov
31.05.2016 15:27, Alex Peshkoff wrote: > But what about AIX or HPUX with native compilers - probably you need to > google carefully re compilers compatibility and tell us how does it look > like to use this feature on them. I understand it's close to unreal to > test all platforms but docs should

Re: [Firebird-devel] Move semantic

2016-05-31 Thread Alex Peshkoff
On 05/31/2016 03:35 PM, Dimitry Sibiryakov wrote: > 31.05.2016 14:20, Dmitry Yemanov wrote: >> Does MSVC10 count as ancient? > For me - yes because it is at least three versions behind of top and, > actually, never > followed standards. > MSDN says that it does support move things in

Re: [Firebird-devel] Move semantic

2016-05-31 Thread Dimitry Sibiryakov
31.05.2016 14:20, Dmitry Yemanov wrote: > Does MSVC10 count as ancient? For me - yes because it is at least three versions behind of top and, actually, never followed standards. MSDN says that it does support move things in contrast to deleted members. I rather worry about exotic

Re: [Firebird-devel] Move semantic

2016-05-31 Thread Dmitry Yemanov
31.05.2016 14:52, Dimitry Sibiryakov wrote: > > The only reason may be support of ancient compilers as it happened with > prohibition of > deleted methods. Does MSVC10 count as ancient? Dmitry -- What NetFlow

Re: [Firebird-devel] Move semantic

2016-05-31 Thread Dimitry Sibiryakov
31.05.2016 13:48, Dmitry Yemanov wrote: > I don't see why we should prohibit > their usage if really necessary. The only reason may be support of ancient compilers as it happened with prohibition of deleted methods. -- WBR, SD.

Re: [Firebird-devel] Move semantic

2016-05-31 Thread Dmitry Yemanov
30.05.2016 18:47, Dimitry Sibiryakov wrote: > > Is it allowed to use move constructor and assignment in Firebird code? AFAIK, we never use them before, but I don't see why we should prohibit their usage if really necessary. Dmitry

[Firebird-devel] Move semantic

2016-05-31 Thread Dimitry Sibiryakov
Hello, All. Is it allowed to use move constructor and assignment in Firebird code? -- WBR, SD. -- What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level.