On Mon, 2016-05-09 at 16:57 +0000, Matthew Brincke wrote:
> I've attached a patch to prevent copying PdfMutex
> instances because POSIX didn't define mutex assignment
> and copying PdfMutex would lead to that AFAIK.

        Hi,
thanks for the patch. Making the two private should make the trick on
its own. Defining a method/constructor which is not implemented is
something I do not like to see. I know it can "more easily" be spot by
the compiler that this method/constructor is used when there is also
declared --no-undefined for the linker (or its equivalent for other
compilers/linkers), but it still feels like a bad practice for me.

I would skip this change altogether, personally.
        Bye,
        zyx 
 
-- 
http://www.litePDF.cz                                 i...@litepdf.cz

------------------------------------------------------------------------------
Mobile security can be enabling, not merely restricting. Employees who
bring their own devices (BYOD) to work are irked by the imposition of MDM
restrictions. Mobile Device Manager Plus allows you to control only the
apps on BYO-devices by containerizing them, leaving personal data untouched!
https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
_______________________________________________
Podofo-users mailing list
Podofo-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/podofo-users

Reply via email to