new raster code

Bernhard Reiter bernhard at intevation.de
Thu Jan 20 20:51:12 CET 2005


On Thu, Jan 20, 2005 at 11:24:16AM -0500, Jonathan Coles wrote:
> Am Donnerstag, den 20.01.2005, 16:00 +0100 schrieb Bernhard Reiter:

> > Can you do me a favour and:
> > 	* split up patches, if possible
> > 	(I understood that removing cpl_mfile.* and bmpdataset.cpp
> > 	is indenpendent? No?)
> 
> done (see attached patch files). you are correct, removing files is a
> seperate issue.
> 
> > 	* diff -u or  diff -c them instead
> > 	* write a Changelog Entry
> 
> done.

Thanks.
I am peeking at it now.

First question:
Why is the CPLError function commented out here?
if ( err )
     {
-        CPLError( CE_Failure, err, "" );
+        //CPLError( CE_Failure, err, "%s\n", CPLGetLastErrorMsg()
);
         return NULL;
     }

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://www.intevation.de/pipermail/thuban-devel/attachments/20050120/24c27d57/attachment.bin


More information about the Thuban-devel mailing list

This site is hosted by Intevation GmbH (Datenschutzerklärung und Impressum | Privacy Policy and Imprint)