diff options
author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2020-02-11 18:47:20 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-02-11 18:47:20 -0800 |
commit | 190433d8150bf719fa0ba972dbacf2214942f54e (patch) | |
tree | 542d09915c36b8e496c0c66950eb0bf29704e271 /Python/pyhash.c | |
parent | 0f0d2e496205c345b182b6572ee09db23f8f9daf (diff) | |
download | cpython-190433d8150bf719fa0ba972dbacf2214942f54e.tar.gz cpython-190433d8150bf719fa0ba972dbacf2214942f54e.zip |
closes bpo-39605: Fix some casts to not cast away const. (GH-18453)
gcc -Wcast-qual turns up a number of instances of casting away constness of pointers. Some of these can be safely modified, by either:
Adding the const to the type cast, as in:
- return _PyUnicode_FromUCS1((unsigned char*)s, size);
+ return _PyUnicode_FromUCS1((const unsigned char*)s, size);
or, Removing the cast entirely, because it's not necessary (but probably was at one time), as in:
- PyDTrace_FUNCTION_ENTRY((char *)filename, (char *)funcname, lineno);
+ PyDTrace_FUNCTION_ENTRY(filename, funcname, lineno);
These changes will not change code, but they will make it much easier to check for errors in consts
(cherry picked from commit e6be9b59a911626d6597fe148c32f0342bd2bd24)
Co-authored-by: Andy Lester <andy@petdance.com>
Diffstat (limited to 'Python/pyhash.c')
-rw-r--r-- | Python/pyhash.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Python/pyhash.c b/Python/pyhash.c index 4c0b929586f..ba224ee3736 100644 --- a/Python/pyhash.c +++ b/Python/pyhash.c @@ -366,7 +366,7 @@ static PyHash_FuncDef PyHash_Func = {fnv, "fnv", 8 * SIZEOF_PY_HASH_T, static uint64_t siphash24(uint64_t k0, uint64_t k1, const void *src, Py_ssize_t src_sz) { uint64_t b = (uint64_t)src_sz << 56; - const uint8_t *in = (uint8_t*)src; + const uint8_t *in = (const uint8_t*)src; uint64_t v0 = k0 ^ 0x736f6d6570736575ULL; uint64_t v1 = k1 ^ 0x646f72616e646f6dULL; |