On 29.08.2011 02:27, Eike Rathke wrote: > Hi Michael, > > On Sunday, 2011-08-28 23:23:03 -0000, [email protected] wrote: > >> URL: http://svn.apache.org/viewvc?rev=1162609&view=rev > >> --- incubator/ooo/trunk/main/autodoc/inc/ary/idl/i_ce2s.hxx (original) >> +++ incubator/ooo/trunk/main/autodoc/inc/ary/idl/i_ce2s.hxx Sun Aug 28 >> 23:23:03 2011 >> @@ -50,6 +50,7 @@ class Ce_2s >> { >> public: >> // LIFECYCLE >> + explicit Ce_2s () { } >> virtual ~Ce_2s(); >> >> static DYN Ce_2s * Create_( > > I took a different approach without introducing that ctor: > https://svn.apache.org/viewvc/incubator/ooo/trunk/main/autodoc/source/ary/idl/i_ce.cxx?r1=1162288&r2=1162554&pathrev=1162554&diff_format=h > > IMHO holding a const dummy just to be able to return a const reference > if the factory didn't yet create an instance is nonsense.
indeed you are right; i just failed to remove this from the patch that i had laying around for weeks (and i had not actually looked at the use site back then which indeed was nonsense).
