Opened 16 years ago

Closed 5 years ago

#2611 closed enhancement (wontfix)

Change GDALRasterBand::TryGetLockedBlockRef() from protected to public

Reported by: lukeroth Owned by: warmerdam
Priority: normal Milestone: closed_because_of_github_migration
Component: GDAL_Raster Version: unspecified
Severity: normal Keywords:
Cc:

Description

Currently, TryGetLockedBlockRef() is a protected method. For interactive display purposes, it would be useful to be able to determine whether a block is already present in the cache before reading, to avoid initiating a possible length read in the main user interface thread. The simplest way that I can see to do this would be to expose TryGetLockedBlockRef() as a public method. This does expose a bit more implementation detail, but other than that I can't see any real drawbacks; I've made the change locally and have not yet run into any issues.

Change History (5)

comment:1 by lukeroth, 16 years ago

Summary: Change TryGetLockedBlockRef() from protected to publicChange GDALRasterBand::TryGetLockedBlockRef() from protected to public

Sorry, I should have been more specific; I'm referring to GDALRasterBand::TryGetLockedBlockRef().

comment:2 by Even Rouault, 15 years ago

Milestone: 1.6.11.7.0

comment:3 by Jukka Rahkonen, 9 years ago

Does the change still feel reasonable to do at some time in the future?

comment:4 by Even Rouault, 9 years ago

Milestone: 1.8.1

Removing obsolete milestone

comment:5 by Even Rouault, 5 years ago

Milestone: closed_because_of_github_migration
Resolution: wontfix
Status: newclosed

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to https://github.com/OSGeo/gdal/issues if it is not already reported there.

Note: See TracTickets for help on using tickets.