Question on php_image_type_to_mime_type

php.internals

Mehdi Achour

23 years ago
Hi all, I'm in php-src/ext/standard/image.c (revision 1.92) Looking at php_image_type_to_mime it seems like there are two constants not handled : IMAGETYPE_JB2 IMAGETYPE_JPX Am I looking at the good version of the C file ? didou

Unnamed Person

23 years ago
"Mehdi Achour" <didou@keliglia.com> a écrit dans le message de news:20030815172728.51523.qmail@pb1.pair.com...
> Hi all,
Yo Mehdi
> > I'm in php-src/ext/standard/image.c (revision 1.92) > Looking at php_image_type_to_mime it seems like there are two constants > not handled : > IMAGETYPE_JB2 > IMAGETYPE_JPX > > Am I looking at the good version of the C file ?
Yes.
> > didou >
-- Regards. M.CHAILLAN Nicolas nicos@php.net www.WorldAKT.com Hébergement de sites internets.

Mehdi Achour

23 years ago
>>I'm in php-src/ext/standard/image.c (revision 1.92) >>Looking at php_image_type_to_mime it seems like there are two constants >>not handled : >> IMAGETYPE_JB2 >> IMAGETYPE_JPX >> >>Am I looking at the good version of the C file ? > > > Yes.
Shouldn't be corrected ? didou

(Marcus Börger)

23 years ago
Hello Mehdi, Friday, August 15, 2003, 10:22:28 PM, you wrote: MA> >>I'm in php-src/ext/standard/image.c (revision 1.92)
>>>Looking at php_image_type_to_mime it seems like there are two constants >>>not handled : >>> IMAGETYPE_JB2 >>> IMAGETYPE_JPX >>> >>>Am I looking at the good version of the C file ? >> >> >> Yes.
MA> Shouldn't be corrected ? No, they are used elsewhere, too. If you mean the support is missing, feel free to add it. Best regards, Marcus mailto:helly@php.net

Mehdi Achour

23 years ago
Marcus Börger wrote:
> Hello Mehdi, > > Friday, August 15, 2003, 10:22:28 PM, you wrote: > > MA> >>I'm in php-src/ext/standard/image.c (revision 1.92) > >>>>Looking at php_image_type_to_mime it seems like there are two constants >>>>not handled : >>>> IMAGETYPE_JB2 >>>> IMAGETYPE_JPX >>>> >>>>Am I looking at the good version of the C file ? >>> >>> >>>Yes. > > > MA> Shouldn't be corrected ? > > No, they are used elsewhere, too. If you mean the support is missing, feel > free to add it. >
Hi Marcus, I don't have karma to php-src :) Here's what should be changed : constant mime-type IMAGETYPE_JPX image/jpx IMAGETYPE_JB2 image/jb2 Can you do it for me ? didou

(Marcus Börger)

23 years ago
Hello Mehdi, Friday, August 15, 2003, 10:47:41 PM, you wrote: MA> Marcus Börger wrote:
>> Hello Mehdi, >> >> Friday, August 15, 2003, 10:22:28 PM, you wrote: >> >> MA> >>I'm in php-src/ext/standard/image.c (revision 1.92) >> >>>>>Looking at php_image_type_to_mime it seems like there are two constants >>>>>not handled : >>>>> IMAGETYPE_JB2 >>>>> IMAGETYPE_JPX >>>>> >>>>>Am I looking at the good version of the C file ? >>>> >>>> >>>>Yes. >> >> >> MA> Shouldn't be corrected ? >> >> No, they are used elsewhere, too. If you mean the support is missing, feel >> free to add it. >>
MA> Hi Marcus, MA> I don't have karma to php-src :) MA> Here's what should be changed : MA> constant mime-type MA> IMAGETYPE_JPX image/jpx MA> IMAGETYPE_JB2 image/jb2 Which source are those based on?
-- Best regards, Marcus mailto:helly@php.net

Mehdi Achour

23 years ago
Marcus Börger wrote:
> Hello Mehdi, > > Friday, August 15, 2003, 10:47:41 PM, you wrote: > > MA> Marcus Börger wrote: > > >>>Hello Mehdi, >>> >>>Friday, August 15, 2003, 10:22:28 PM, you wrote: >>> >>>MA> >>I'm in php-src/ext/standard/image.c (revision 1.92) >>> >>> >>>>>>Looking at php_image_type_to_mime it seems like there are two constants >>>>>>not handled : >>>>>> IMAGETYPE_JB2 >>>>>> IMAGETYPE_JPX >>>>>> >>>>>>Am I looking at the good version of the C file ? >>>>> >>>>> >>>>>Yes. >>> >>> >>>MA> Shouldn't be corrected ? >>> >>>No, they are used elsewhere, too. If you mean the support is missing, feel >>>free to add it. >>> > > > MA> Hi Marcus, > > MA> I don't have karma to php-src :) > MA> Here's what should be changed : > > MA> constant mime-type > > MA> IMAGETYPE_JPX image/jpx > MA> IMAGETYPE_JB2 image/jb2 > > Which source are those based on? > >
JB2 : http://cgi.netscape.com/cgi-bin/pi_moreinfo.cgi?PID=11473 JPX : http://www.ietf.org/internet-drafts/draft-singer-jp2-01.txt The source for JB2 mime type may seems not enough. didou

(Marcus Börger)

23 years ago
Hello Mehdi, Friday, August 15, 2003, 11:20:52 PM, you wrote: MA> Marcus Börger wrote:
>> Hello Mehdi, >> >> Friday, August 15, 2003, 10:47:41 PM, you wrote: >> >> MA> Marcus Börger wrote: >> >> >>>>Hello Mehdi, >>>> >>>>Friday, August 15, 2003, 10:22:28 PM, you wrote: >>>> >>>>MA> >>I'm in php-src/ext/standard/image.c (revision 1.92) >>>> >>>> >>>>>>>Looking at php_image_type_to_mime it seems like there are two constants >>>>>>>not handled : >>>>>>> IMAGETYPE_JB2 >>>>>>> IMAGETYPE_JPX >>>>>>> >>>>>>>Am I looking at the good version of the C file ? >>>>>> >>>>>> >>>>>>Yes. >>>> >>>> >>>>MA> Shouldn't be corrected ? >>>> >>>>No, they are used elsewhere, too. If you mean the support is missing, feel >>>>free to add it. >>>> >> >> >> MA> Hi Marcus, >> >> MA> I don't have karma to php-src :) >> MA> Here's what should be changed : >> >> MA> constant mime-type >> >> MA> IMAGETYPE_JPX image/jpx >> MA> IMAGETYPE_JB2 image/jb2 >> >> Which source are those based on? >> >>
MA> JB2 : http://cgi.netscape.com/cgi-bin/pi_moreinfo.cgi?PID=11473 MA> JPX : http://www.ietf.org/internet-drafts/draft-singer-jp2-01.txt Both are not official and the ietf draft was declined (it's outdated and didn't make it as a rfc). So i don't really see why i should change the current behavior and implement something non standard.
-- Best regards, Marcus mailto:helly@php.net

Mehdi Achour

23 years ago
Marcus BöRger wrote:
> Hello Mehdi, > > Friday, August 15, 2003, 11:20:52 PM, you wrote: > > MA> Marcus Börger wrote: > > >>>Hello Mehdi, >>> >>>Friday, August 15, 2003, 10:47:41 PM, you wrote: >>> >>>MA> Marcus Börger wrote: >>> >>> >>> >>>>>Hello Mehdi, >>>>> >>>>>Friday, August 15, 2003, 10:22:28 PM, you wrote: >>>>> >>>>>MA> >>I'm in php-src/ext/standard/image.c (revision 1.92) >>>>> >>>>> >>>>> >>>>>>>>Looking at php_image_type_to_mime it seems like there are two constants >>>>>>>>not handled : >>>>>>>>IMAGETYPE_JB2 >>>>>>>>IMAGETYPE_JPX >>>>>>>> >>>>>>>>Am I looking at the good version of the C file ? >>>>>>> >>>>>>> >>>>>>>Yes. >>>>> >>>>> >>>>>MA> Shouldn't be corrected ? >>>>> >>>>>No, they are used elsewhere, too. If you mean the support is missing, feel >>>>>free to add it. >>>>> >>> >>> >>>MA> Hi Marcus, >>> >>>MA> I don't have karma to php-src :) >>>MA> Here's what should be changed : >>> >>>MA> constant mime-type >>> >>>MA> IMAGETYPE_JPX image/jpx >>>MA> IMAGETYPE_JB2 image/jb2 >>> >>>Which source are those based on? >>> >>> > > > MA> JB2 : http://cgi.netscape.com/cgi-bin/pi_moreinfo.cgi?PID=11473 > MA> JPX : http://www.ietf.org/internet-drafts/draft-singer-jp2-01.txt > > Both are not official and the ietf draft was declined (it's outdated and > didn't make it as a rfc). So i don't really see why i should change the > current behavior and implement something non standard.
Ok, so lest's forget about it ;) Sorry for the time wasted. Mehdi