On Wed, 9 May 2007, Petr Cerny wrote:

> > missing #include <malloc.h>
> 
> If you don't like malloc.h, probably posix_memalign() could be of use.

        If you look more closely at the code you will see that the 
        reference to memalign() (for which malloc.h is needed) is in the
        posix_memalign() error handling.  If posix_memalign() fails (or is
        one of the broken implemtations) then memalign() is used.

        Changes have been made to check for malloc.h and conditionally
        include malloc.h if it exists (it does not exist on all systems).

> if( qtchroma == QUICKTIME_YUV420 ){...}
> 
> Iit might be better to use:
> if(strcmp(qtchroma,QUICKTIME_YUV420)==0){...}

        That will work since earlier usage of 'qtchroma' was 

                qtchroma = QUICKTIME_YUV420; 

> (although under some conditions it will have the desired effect).

        True.  Usually that is a bug - makes sense to change it.

        Steven Schultz


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Mjpeg-users mailing list
Mjpeg-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mjpeg-users

Reply via email to