Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#6853 closed defect (fixed)

GRASS G_set_window => Rast_set_window

Reported by: hcho Owned by: warmerdam
Priority: normal Milestone:
Component: default Version: unspecified
Severity: normal Keywords:
Cc: martinl

Description

The GRASS GIS driver needs to be updated so that it calls Rast_[gs]et_window instead of G_[gs]et_window for GRASS 7. Please refer to the attached patch.

Attachments (1)

grass57dataset.cpp.patch​ (485 bytes ) - added by hcho 7 years ago.
Patch for GRASS 7

Download all attachments as: .zip

Change History (6)

by hcho, 7 years ago

Attachment: grass57dataset.cpp.patch​ added

Patch for GRASS 7

comment:1 by Even Rouault, 7 years ago

From which GRASS version is this needed ? I think the current driver compiles fine with 7.0

comment:2 by hcho, 7 years ago

It compiles fine with 7 because there are both G_[gs]et_window and Rast_[gs]et_window, but the Rast_ version was added in 7 to handle the window for raster reading/writing. This issue was discussed before in ​https://trac.osgeo.org/gdal/ticket/2953#comment:26. G_set_window doesn't change the window for raster reading and can produce an image outside the requested region.

Last edited 7 years ago by hcho (previous) (diff)

comment:3 by Even Rouault, 7 years ago

Cc: martinl added

Martin, any opinion on the above ?

comment:4 by Even Rouault, 7 years ago

Resolution: → fixed
Status: new → closed

In 37964:

GRASS: use Rast_get_window/Rast_set_window for GRASS 7 (patch by hcho, fixes #6853)

comment:5 by Even Rouault, 7 years ago

I've committed the patch in trunk. Would probably deserve a backport in 2.1.3 but I'd prefer having confirmation that's OK.

Note: See TracTickets for help on using tickets.