Commit Graph

71 Commits

Author SHA1 Message Date
Wez Furlong
659a071e3d Streams are all tracked as resources now.
Add some logic that will help track down leaks
when debug is enabled.
2002-03-20 14:21:30 +00:00
Wez Furlong
c9d5e13738 TSRMLS related work on streams, as discussed with Zeev.
# Should be the last "broad" commit for a while
# Don't forget to make clean ; make
2002-03-18 18:54:32 +00:00
Marcus Boerger
68f4e7c328 -streams are no longer an option 2002-03-18 05:32:25 +00:00
Marcus Boerger
c645babd8a -use php_stream_make_seekable 2002-03-18 00:12:47 +00:00
Wez Furlong
d2d8762267 Streams now make more use of the memory manager, so tracking down
leaking streams should be easier.
# I hate these big commits
2002-03-17 22:50:59 +00:00
Marcus Boerger
20f2e63289 #removed code to track down error in streams 2002-03-17 12:16:29 +00:00
Marcus Boerger
bcad7a593a -quick hack to bug in streams....but where? 2002-03-16 21:19:17 +00:00
Marcus Boerger
d50102b6ab -disabling debug
#Wez: i have an error when closing the original file after stream copy...
2002-03-16 20:25:51 +00:00
Marcus Boerger
fe9cb020ae -fix bug for not correct terminated comments
-switch to streams
@exif functions now support streams using memory streams where @necessary (marcus)
2002-03-16 20:12:17 +00:00
Wez Furlong
0f65280cb5 New PHP streams... 2002-03-15 21:03:08 +00:00
Marcus Boerger
e03a8a995c -handling 2002-03-13 00:21:51 +00:00
Marcus Boerger
3c47fb91e8 -usability
-tests
2002-03-12 16:43:29 +00:00
Marcus Boerger
8a622d0668 -return size of thumbnail even if it not read 2002-03-12 13:49:40 +00:00
Marcus Boerger
06a242169c -support for array tags
-better memory handling
-support for thumbnails in TIFF format
-new functions exif_imagetype
@-new function exif_imagetype
@-exif_thumbnail fupports thumbnails in TIFF format and can return size
@ of thumbnail
2002-03-12 06:14:45 +00:00
Marcus Boerger
42e478c7aa -support for out of section data 2002-03-08 20:56:44 +00:00
Marcus Boerger
27242a7a46 -change IsColor from non zero to 1
-new test and example for documentation
2002-03-08 11:33:46 +00:00
Marcus Boerger
9363bfad7e -Support for IFD Arrays
#necessary for update/write operations
2002-03-08 01:09:54 +00:00
Marcus Boerger
d994d0b47d TIFF support for 'non' digicam files
# same as in standard/image.c but here it's more complex
2002-03-08 01:06:59 +00:00
foobar
88750086cc - Fixed some compile warnings and removed bogus comments. 2002-03-07 21:41:51 +00:00
Marcus Boerger
0a0942b193 -comments
#just another jhead.c line
2002-03-06 20:52:17 +00:00
Marcus Boerger
15d74e343b -comments 2002-03-06 20:24:29 +00:00
Marcus Boerger
0b5759f7bf -missing efree
-incorrect index/length computing: left from jhead
#no errors left with test files - controlled manually with hexeditor
2002-03-06 18:11:57 +00:00
Marcus Boerger
0ecd2f0876 -new working thumbnail code
-everything uses new data structures
-new function exif_thumbnail
#nearly stripped off all jhead code
@read_exif_data is now an alias for exif_read_data
@  important to differenciate old/new version for tests)
@new function exif_tagname returns the names of tags
@new function exif_thumbnail extracts embedded thumbnail (Marcus)
#switched to internal version 1.2
2002-03-06 09:31:32 +00:00
Marcus Boerger
088450efce -fixes
-changed internal data structures
#nearly stripped off all jhead code
2002-03-05 19:47:19 +00:00
Marcus Boerger
6b2a074330 - Read IsColor for TIFF 2002-03-04 23:19:27 +00:00
Marcus Boerger
a7403576e1 - disabled debug information
- added missing width/height from primary ifd for TIFF
2002-03-04 17:09:24 +00:00
Marcus Boerger
be5908b06c -addition to last change: php_image.h must be included now 2002-03-04 09:29:06 +00:00
Rasmus Lerdorf
83e6dbcf19 Let getimagesize() and read_exif_data() share the same constants to
promote a little bit of code reuse here on two very similar problems.
2002-03-04 07:21:30 +00:00
Marcus Boerger
642ab24df1 -Added TIFF support
-Changed parameters after checking bugdatabase and discussion with
    Rasmus: 1st=Filename, 2nd=NeededSections
# 3rd=Thumbnail, 4th=All
#   3rd and 4th parameter are currently diabled because the code
#   does not work correctly (will correct this in later versions).
#   Changing the parameters will not cause any problems because the
#   old parameter functions never found their way in documentation
#   and the they did not work...
@-Added TIFF support
@-Changed parameters after checking bugdatabase and discussion with Rasmus
@    1st=Filename, 2nd=NeededSections
@    If the 2nd parameter is not present or 0 or '' the function will return
@        a result array even if no data was read from file.
#        Personally i consider a default behaviour to generate no array if
#        no data is present as a bad idea because here we can generate as
#        much information as possible....
@    If the 2nd parameter is a string then that string describes which
@        sections have to be present in the file to generate a result array.
@        If none of the needed section is found the result will be false.
@        Section strings are "ANY_TAG","IFD0","COMMENT","EXIF","GPS","INTEROP"
@        "FPIX","APP12". Each string has to be separated by a colon. So
@        "COMMENT,EXIF" returns an array if either a comment or an Exif
@        section is present. "IFD0" is standard datastructure in a TIFF/JPEG
@        file that contains image information. "EXIF" is the Exif structure
@        of TIFF/JPEG (IFD0 subdirectory). "GPS", "INTEROP", "FPIX", "APP12"
@        are additional IFD0 subdirectories. The APP12 support is an
@		 experimental Olympus support.
#        Do not worry this is no problem, it is safe.
@(Marcus)
2002-03-04 06:21:58 +00:00
Marcus Boerger
442773abb3 Wrong brackets by rasmus...but sorry the fault was mine because i missed them...
i also decided to call the first working version 1.0a

by the way: currently i am working on tiff files and it looks like it may work
2002-03-02 15:43:19 +00:00
Rasmus Lerdorf
15ba6402bd Fix a couple of warnings 2002-03-02 08:37:28 +00:00
Marcus Boerger
0857bdbcde +Support for Photographer/Editor Copyright as associative array as this is a new feature the change (optionally being an array) has to be mentioned in documentation.
+New function exif_headername can be used to read the internal Tag namelist (was mainly created for debugging purpose but maybe somone writes code to create/update exif headers here).
+An internal version number is present.
+A testpage is supplied test.txt describes how the test works.
+The oldfunction read_exif_data has got an alias exif_read_data

As the old version of this module is very buggy i decided to implement the testpage (test.txt) and to create the alias. The test script only works with the alias as the old version does not pass tests. By the way it seems a good way to prepend 'exif_' to all functions in the module.
2002-03-01 04:01:26 +00:00
Marcus Boerger
05b13da525 Changed file to match CODING_STANDARDS except function names that match jhead.c project. I think it is acceptable using naming conventions of other projects when borrowing code. 2002-02-28 12:20:17 +00:00
Sebastian Bergmann
7d983e5faf Fix headers. 2002-02-28 06:26:50 +00:00
Marcus Boerger
98ebd1c28b +Added UNICODE support for Comments
+Added Description,Artist
+Added missing memory deallocation
+Corrected error with multiple comments
+Corrected handling of ExifVersion, Tag has 4 ASCII characters *WITHOUT* NUL
+Corrected handling of Thumbnailsize if current source detects size < 0
+Changed all fields to char* that do not have a maximum length in EXIF standard
+Undocumented second Parameter ReadAll frees memory to early -> moved to third position default changed to false -> faster
+New second Parameter [true|false] to specify whether or not to to read thumbnails -> reading is timeconsumpting suppose default should be false -> done so
2002-02-28 01:16:27 +00:00
Sebastian Bergmann
38933514e1 Update headers. 2001-12-11 15:32:16 +00:00
Stig Bakken
689252082c * zend_module_entry change: apino, debug and zts are moved first,
see README.EXTENSIONS file for upgrade help.
@Introduced extension version numbers (Stig)
2001-10-11 23:33:59 +00:00
Jeroen van Wolffelaar
c033288573 Back-substitute for Z_* macro's. If it breaks some extension (the script isn't optimal, it parses for example var->zval.value incorrect) please let me know. 2001-09-25 21:58:48 +00:00
Derick Rethans
78747bd2df - Don't wrap lines... this is annoying while coding. 2001-09-09 13:29:31 +00:00
foobar
5296d29b59 Killed a compile warning. 2001-08-22 08:34:04 +00:00
Rasmus Lerdorf
b732d8da35 Missing define 2001-08-16 23:31:54 +00:00
Rasmus Lerdorf
2b4c2a2ff5 Commit a bit of random plane hacking.
- A slight exif fix from the latest version of jhead
 - Make ext_skel's confirm_module_compiled function consistent
2001-08-16 00:16:00 +00:00
Zeev Suraski
f6f6c4d7e6 Whitespace 2001-08-11 16:39:07 +00:00
Frank M. Kromann
57946eb3be Adding missing TSRM macros.
Kill signed/unsigned compiler warning
2001-08-06 16:58:27 +00:00
Rasmus Lerdorf
628c21275f support for multiple exif comments
@ - Enhance read_exif_data() to support multiple comment tags (Rasmus)
2001-08-04 22:35:03 +00:00
Rasmus Lerdorf
cc40cdc7d7 Fix newly introduced crash on a file-not-found 2001-08-04 18:08:13 +00:00
foobar
a9f850f34e fixed a memleak. 2001-08-04 03:06:12 +00:00
foobar
faaeee3705 Fixed bug: #11784 (combined with fixes Rasmus committed..) 2001-08-04 01:56:40 +00:00
Rasmus Lerdorf
9b0df45da7 Change from E_ERROR to E_WARNING in the exif extension
@ Change from E_ERROR to E_WARNING in the exif extension (Rasmus)
2001-08-03 22:33:10 +00:00
Zeev Suraski
797a079a95 More TSRMLS_FETCH work, and a bit of cleanup 2001-07-30 06:18:13 +00:00