Opened 16 years ago
Closed 10 years ago
#413 closed defect (fixed)
doxygen comments in get_row.c are wrong?
Reported by: | karme | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | 6.4.4 |
Component: | LibGIS | Version: | svn-releasebranch64 |
Keywords: | G_get_raster_row(), doxygen | Cc: | |
CPU: | All | Platform: | All |
Description
Either the code in get_row.c is wrong or the the doxygen strings. G_get_raster_row does not return 0 if the row is outside the window.
Attached a patch trying to fix this. Note: this was a quick hack I tested while trying to speed up d.rast => only supplied to explain the problem.
Attachments (1)
Change History (7)
by , 16 years ago
Attachment: | outofwindow.diff added |
---|
comment:1 by , 12 years ago
Milestone: | 6.4.0 → 6.4.3 |
---|---|
Version: | unspecified → svn-releasebranch64 |
follow-up: 4 comment:2 by , 12 years ago
Replying to karme:
Either the code in get_row.c is wrong or the the doxygen strings.
The doxygen strings are wrong.
G_get_raster_row does not return 0 if the row is outside the window.
There's no reason for higher-level code to care about what is essentially an implementation detail. It shouldn't matter whether the row is all-nulls because it's outside the array or because the stored data happens to be all-nulls.
comment:3 by , 11 years ago
Component: | Default → LibGIS |
---|---|
CPU: | Unspecified → All |
Keywords: | G_get_raster_row() doxygen added |
Milestone: | 6.4.3 → 6.4.4 |
Platform: | Unspecified → All |
follow-up: 5 comment:4 by , 10 years ago
Replying to glynn:
Replying to karme:
Either the code in get_row.c is wrong or the the doxygen strings.
The doxygen strings are wrong.
G_get_raster_row does not return 0 if the row is outside the window.
There's no reason for higher-level code to care about what is essentially an implementation detail. It shouldn't matter whether the row is all-nulls because it's outside the array or because the stored data happens to be all-nulls.
IIUC, return values are either 1 or -1, so just erasing the line in the comments mentioning "return 0" should be enough ?
follow-up: 6 comment:5 by , 10 years ago
Replying to mlennert:
IIUC, return values are either 1 or -1, so just erasing the line in the comments mentioning "return 0" should be enough ?
Right.
The internal functions compute_window_row() and get_map_row_nomask() return 0 if the row is outside of the map's data, but that detail doesn't propagate up to any of the documented public functions.
comment:6 by , 10 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Replying to glynn:
Replying to mlennert:
IIUC, return values are either 1 or -1, so just erasing the line in the comments mentioning "return 0" should be enough ?
Right.
The internal functions compute_window_row() and get_map_row_nomask() return 0 if the row is outside of the map's data, but that detail doesn't propagate up to any of the documented public functions.
Correction of doxygen comment committed to devel6 (r60143) and rel64 (r60144).
Closing the bug.
Moritz
Devs: any opinion?