core
/
eccodes
mirror of https://github.com/ecmwf/eccodes.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #24 from mkrupcale/encode-unsigned-long-pointer-increment 

bits: grib_encode_unsigned_long: increment pointer, not the pointed-to value
This commit is contained in:
shahramn 2020-03-17 18:12:04 +00:00 committed by GitHub
parent 512715df24 57c3d8fa8b
commit a2d3d57789
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/grib_bits_any_endian.c
View File

@ -280,12 +280,7 @@ int grib_encode_unsigned_long(unsigned char* p, unsigned long val, long* bitp, l
else { else {
tmp = ((val >> len) | ((*p) & dmasks[n])); tmp = ((val >> len) | ((*p) & dmasks[n]));
} }
*p = tmp; *p++ = tmp;
(*p)++;
/*Beware of code like this! compiler warning: operation may be undefined
Read GCC manual on -Wsequence-point*/
/* *p = ((val << -len) | ((*p)++ & dmasks[n])); */
} }
/* write the middle words */ /* write the middle words */

3
src/grib_bits_ibmpow.c
View File

@ -107,8 +107,7 @@ int grib_encode_unsigned_long(unsigned char* p, unsigned long val, long* bitp, l
else { else {
tmp = ((val >> len) | ((*p) & dmasks[n])); tmp = ((val >> len) | ((*p) & dmasks[n]));
} }
*p = tmp; *p++ = tmp;
(*p)++;
} }
/* write the middle words */ /* write the middle words */