Even Rouault
Even Rouault
> but it would be nice for developers to have it on the build page. can you contribute a document enhancement for that?
> Thus the extra test is quite expensive I haven't measured the perf impact, but the only significant impact I would anticipate would only be in loops where you would...
Closing, as superseded per https://github.com/OSGeo/gdal/pull/9285
https://github.com/OSGeo/gdal/issues/3060 discussed that (more specifically for Linux), but my thoughts in https://github.com/OSGeo/gdal/issues/3060#issuecomment-719794405 also apply
> Would it be reasonable for GDAL to be tolerant of these incoherences? Could make sense. The driver, as currently written due to the hypothesis taken on the file structure,...
Code formatting issues: ```If you are seeing this message in CI, reproduce locally with: `pre-commit run --all-files`. To run `pre-commit` as part of git workflow, use `pre-commit install`. ``` Cf...
some linting issues remaining: https://github.com/OSGeo/gdal/actions/runs/7308961912/job/19916097431?pr=8995 cf my comment https://github.com/OSGeo/gdal/pull/8995#issuecomment-1868266258 how to solve them automatically
Things look good to me. Hopefully @landam can give it some review too
OK, I'm merging this... There are likely potential optimizations like the one I suggested in https://github.com/OSGeo/gdal/pull/8995#pullrequestreview-1866404386 , but this should be good enough as it for now.
Apparently, according to https://curl.se/libcurl/c/CURLOPT_RANGE.html, sftp:// should support range requests, so that's a good precondition. Why it doesn't work? I don't know. Would require debugging... Well possible that there are some...