D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 6820 - etc.c.curl missing const
Summary: etc.c.curl missing const
Status: RESOLVED FIXED
Alias: None
Product: D
Classification: Unclassified
Component: phobos (show other issues)
Version: D2
Hardware: All All
: P2 normal
Assignee: No Owner
URL:
Keywords: bootcamp
Depends on:
Blocks:
 
Reported: 2011-10-16 19:50 UTC by Brad Roberts
Modified: 2017-12-18 22:56 UTC (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description Brad Roberts 2011-10-16 19:50:21 UTC
Many of the curl api's take const buffers/strings but the .d wrapper leaves out the const'ness.  For example:

curl.h:
CURL_EXTERN char *curl_easy_unescape(CURL *handle,
                                     const char *string,
                                     int length,
                                     int *outlength);

curl.d:
char * curl_easy_unescape(CURL *handle, char *string, int length, int *outlength);
Comment 1 github-bugzilla 2017-11-04 16:41:45 UTC
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/a516b51d0e44316f6371a3cd78bd3133a035e99b
Fix Issue 6820 - etc.c.curl missing const

https://github.com/dlang/phobos/commit/25a6420e2bc277c75c934e14adfdf5951797e1d8
Merge pull request #5843 from Darredevil/issue-6820-curl-missing-const

Fix Issue 6820 - etc.c.curl missing const
merged-on-behalf-of: Vladimir Panteleev <github@thecybershadow.net>
Comment 2 github-bugzilla 2017-12-18 22:56:22 UTC
Commits pushed to stable at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/a516b51d0e44316f6371a3cd78bd3133a035e99b
Fix Issue 6820 - etc.c.curl missing const

https://github.com/dlang/phobos/commit/25a6420e2bc277c75c934e14adfdf5951797e1d8
Merge pull request #5843 from Darredevil/issue-6820-curl-missing-const