[poppler] [PATCH] add "const" typing to Error handling

Albert Astals Cid aacid at kde.org
Mon Apr 21 10:14:44 PDT 2008


A Dilluns 21 Abril 2008, Carl Worth va escriure:
> On Sun, 20 Apr 2008 16:03:42 +0200, Albert Astals Cid wrote:
> > > constant to 'char*' SplashOutputDev.cc:1042: warning: deprecated
> > > conversion from string constant to 'char*' SplashOutputDev.cc:1053:
> > > warning: deprecated conversion from string constant to 'char*' ...
> >
> > poppler code is full of those "char *" instead of "const char *" usage,
> > we are not patching them because it will make merging with xpdf code
> > harder. That's why we use -Wno-write-strings by default to not see those
> > warnings.
>
> I don't follow.
>
> If there's a code improvement here, is there any reason not to push
> this up to the xpdf code as well?

And how do you suggest us to do this? If we could push things into xpdf, 
poppler would not exist at all.

> And is "merge with xpdf" intended to be a regular activity for the
> foreseeable future? 

Yes, each time the xpdf author decides to release a new xpdf version we plan 
to merge the changes.

> Is there any effort to get xpdf and poppler 
> unforked, (that is, to make xpdf depend on poppler)?

xpdf author was never interested in collaboration, i don't see what can have 
changed over time.

Albert

>
> -Carl




More information about the poppler mailing list