Re[2]: [PECL-CVS] cvs: pecl /pdf package.xml pdf.c pdf.dsp pdf4.h php_pdf.h php_pdf4.h
| From: | Derick Rethans | Date: | Sun, 14 Nov 2004 20:33:53 +0000 |
| Subject: | Re[2]: [PECL-CVS] cvs: pecl /pdf package.xml pdf.c pdf.dsp pdf4.h php_pdf.h php_pdf4.h | ||
| References: | 1 2 3 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-13851@lists.php.net to get a copy of this message | ||
On Sun, 14 Nov 2004, Rainer Schaaf wrote:
> Hello Derick,
>
> Sunday, November 14, 2004, 9:26:13 PM, you wrote:
>
> > On Sun, 14 Nov 2004, Rainer Schaaf wrote:
>
> >> rjs Sun Nov 14 15:20:30 2004 EDT
> >>
> >> Added files:
> >> /pecl/pdf pdf4.h php_pdf4.h
>
> > Why did you put code in a .h file?
>
> Because it is included in the pdf.c Source file. So I thought it is
> better make it an .h file.
I think it's a large "WTF" factor here ;-) I also wonder why it is
included BTW, can't you just tune the config.m4 file to only compile the
file that you really need?
regards,
Derick
--
Derick Rethans
http://derickrethans.nl | http://ez.no | http://xdebug.org