Hi zyx,

I couldn't really find it right now searching the Web,
but AFAIK this is a known idiom (short of making a class
with only static methods a namespace instead, which I've
found as a recommendation, but which I think is too radical
and would probably wreak havoc with the doxygen-generated
documentation), preventing the compiler from automatically
generating a public default constructor by declaring one
yourself, but private. So the class can't be instantiated
from the outside (except if it's an inner class maybe).

I have left off the implementation to also prevent it from
being instantiated from the inside (as IMHO there's really
no need to exist for any such instances).

The build errors have disappeared with the patch which now
is committed as svn r1697 when not cross-building. I mean
that with the patch here under discussion applied, so it is
build-tested on GNU/Linux. But when cross-building to Win32
they still occur, even without the patch applied. They also
don't say anything about the factory classes, so I don't think
they're related. Please don't revert any commits, I'll investigate
the failures soon, just not this Monday because I'll have quite a
bit of job work. Otherwise, please do all what you can answer to
yourself/the project for.


Best regards, mabri




----- Original Message -----
From: zyx <z...@litepdf.cz>
To: podofo-users@lists.sourceforge.net
Sent: 18:50 Sunday, 29 November 2015
Subject: Re: [Podofo-users] Prohibit instantiation of factories (all methods 
static)

On Fri, 2015-11-20 at 23:41 +0000, Matthew Brincke wrote:
> a patch for that mentioned in the subject
> is attached to this e-mail.

    Hi,
what am I missing here? Those are just default constructors, created by
the compiler; except of the one added to the private sections. The
other odd thing is that you define a "function", but you do not
implement it. You said something about missing symbols, right?




> Please review,
> and if accepted, please commit (separately)
> to the public repository, otherwise please
> tell me on-list why not.

No need to write this to each email, it's understood on its own. :)
    Bye,
    zyx

-- 
http://www.litePDF.cz                                 i...@litepdf.cz



------------------------------------------------------------------------------
_______________________________________________
Podofo-users mailing list
Podofo-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/podofo-users

------------------------------------------------------------------------------
_______________________________________________
Podofo-users mailing list
Podofo-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/podofo-users

Reply via email to