Re: [gdal-dev] Re: Compiling in Embarcadero's CBuilder

2010-04-21 Thread Philippe Vachon
Hi,

I've committed a fix (r19493) that will disable the use of the new 
GDALCopyWords implementation. The old and new implementations are functionally 
equivalent, but the new implementation is in some cases 5-10x faster than the 
old implementation. However, there is no problem with disabling it and 
affecting the functionality of GDAL.

For future reference, if you hit a bug in GDAL, one of the best ways to get 
developer attention is to file a bug report on the GDAL Trac website 
(http://trac.osgeo.org/gdal/newticket), but this first requires creating an 
OSGeo account at http://www.osgeo.org/osgeo_userid.

Cheers,
Philippe


On 2010-04-20, at 4:34 PM, jcurru wrote:

 
 Thanks for your answer, Philippe.
 I could make rasterio.cpp compile by making the following change (line 38):
 
 // For now, work around MSVC++ 6.0's broken template support. If this value
 // is not defined, the old GDALCopyWords implementation is used.
 #define USE_NEW_COPYWORDS 1
 #endif
 #undef USE_NEW_COPYWORDS // changed
 
 I guess gdal will use some kind of former strategie to copy bytes.
 I hope this will generate valid code.
 
 But you are telling me to file a ticket on GDAL's trac
 (http://trac.osgeo.org/gdal) and assign it to me (pvachon). Due to my poor
 english and lack of experience in this forum, I unfortunately don't know
 what you mean. Would you be so kind to explain it to me?
 
 Thanks again.
 -- 
 View this message in context: 
 http://n2.nabble.com/Compiling-in-Embarcadero-s-CBuilder-tp4930084p4933551.html
 Sent from the GDAL - Dev mailing list archive at Nabble.com.
 ___
 gdal-dev mailing list
 gdal-dev@lists.osgeo.org
 http://lists.osgeo.org/mailman/listinfo/gdal-dev

___
gdal-dev mailing list
gdal-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/gdal-dev


[gdal-dev] Re: Compiling in Embarcadero's CBuilder

2010-04-20 Thread jcurru

Thanks for your answer, Philippe.
I could make rasterio.cpp compile by making the following change (line 38):

// For now, work around MSVC++ 6.0's broken template support. If this value
// is not defined, the old GDALCopyWords implementation is used.
#define USE_NEW_COPYWORDS 1
#endif
#undef USE_NEW_COPYWORDS // changed

I guess gdal will use some kind of former strategie to copy bytes.
I hope this will generate valid code.

But you are telling me to file a ticket on GDAL's trac
(http://trac.osgeo.org/gdal) and assign it to me (pvachon). Due to my poor
english and lack of experience in this forum, I unfortunately don't know
what you mean. Would you be so kind to explain it to me?

Thanks again.
-- 
View this message in context: 
http://n2.nabble.com/Compiling-in-Embarcadero-s-CBuilder-tp4930084p4933551.html
Sent from the GDAL - Dev mailing list archive at Nabble.com.
___
gdal-dev mailing list
gdal-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/gdal-dev