On Mon, Jul 4, 2011 at 9:16 AM, Alex Mantaut <alexmant...@suremptec.com.ar> wrote: > Hi Frank, > In that case all the handling for the CSK product should go on the > Open() method from hdf5imagedataset? Is there any way to avoid polluting the > hdf5imagedataset with all the subproduct's constants?
Generally I would just use the constants inline rather than predefine them for this purpose. If you find the product stuff is getting really bulky you could move it into a separate source file, but I'd suggest avoiding extra specific classes. > Shall I define Product > specific private methods as: getCSKGeotransform() to get the information? > regards Normally in the open we would do a product-specific method to fetch metadata and it would set the metadata data members, geotransform and other similar stuff rather than having many product specific methods for particular data items. I can try to review your patch when ready. Best regards, -- ---------------------------------------+-------------------------------------- I set the clouds in motion - turn up | Frank Warmerdam, warmer...@pobox.com light and sound - activate the windows | http://pobox.com/~warmerdam and watch the world go round - Rush | Geospatial Programmer for Rent _______________________________________________ gdal-dev mailing list gdal-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/gdal-dev
