Opened 17 years ago
Closed 14 years ago
#1743 closed enhancement (wontfix)
Const correctness in cpl_http module
| Reported by: | Mateusz Łoskot | Owned by: | Mateusz Łoskot |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | default | Version: | svn-trunk |
| Severity: | normal | Keywords: | const-correctness http |
| Cc: |
Description
AFAIR, the cpl_http.h|cpp module is quite new and not widely used across GDAL components. I'd suggest to review it and fix const-correctness of input parameters. For instance, function CPLHTTPFetch takes char papszOptions but this parameters is (conceptually) read-only. So, my suggestion is to change the prototype of CPLHTTPFetch to:
CPLHTTPResult CPLHTTPFetch(const char* pszUrl, const char* const* papszOptions);
Here is an example of potential problems related to the issue reported in this ticket.
Change History (2)
comment:1 by , 14 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 14 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.
No feedback, no interest, giving up.