mirrors/FreeRDP
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Corrected const arguments for wStream functions

Browse Source 
Functions only extracting values now use const for wStream argument
This commit is contained in:
Armin Novak 2022-01-31 10:29:41 +01:00 committed by akallabeth
parent 6fc260b285
commit bc08ebc01c
2 changed files with 37 additions and 33 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
winpr/include/winpr/stream.h
View File

@ -50,9 +50,9 @@ extern "C"
}; };
typedef struct _wStream wStream; typedef struct _wStream wStream;
static INLINE size_t Stream_Capacity(wStream* _s); static INLINE size_t Stream_Capacity(const wStream* _s);
WINPR_API size_t Stream_GetRemainingCapacity(wStream* _s); WINPR_API size_t Stream_GetRemainingCapacity(const wStream* _s);
WINPR_API size_t Stream_GetRemainingLength(wStream* _s); WINPR_API size_t Stream_GetRemainingLength(const wStream* _s);
WINPR_API BOOL Stream_EnsureCapacity(wStream* s, size_t size); WINPR_API BOOL Stream_EnsureCapacity(wStream* s, size_t size);
WINPR_API BOOL Stream_EnsureRemainingCapacity(wStream* s, size_t size); WINPR_API BOOL Stream_EnsureRemainingCapacity(wStream* s, size_t size);
@ -141,11 +141,13 @@ extern "C"
{ \ { \
WINPR_ASSERT(_s); \ WINPR_ASSERT(_s); \
WINPR_ASSERT(Stream_GetRemainingLength(_s) >= 8); \ WINPR_ASSERT(Stream_GetRemainingLength(_s) >= 8); \
(_v) = (_t)( \ (_v) = (_t)((UINT64)(*(_s)->pointer) + (((UINT64)(*((_s)->pointer + 1))) << 8) + \
(UINT64)(*(_s)->pointer) + (((UINT64)(*((_s)->pointer + 1))) << 8) + \ (((UINT64)(*((_s)->pointer + 2))) << 16) + \
(((UINT64)(*((_s)->pointer + 2))) << 16) + (((UINT64)(*((_s)->pointer + 3))) << 24) + \ (((UINT64)(*((_s)->pointer + 3))) << 24) + \
(((UINT64)(*((_s)->pointer + 4))) << 32) + (((UINT64)(*((_s)->pointer + 5))) << 40) + \ (((UINT64)(*((_s)->pointer + 4))) << 32) + \
(((UINT64)(*((_s)->pointer + 6))) << 48) + (((UINT64)(*((_s)->pointer + 7))) << 56)); \ (((UINT64)(*((_s)->pointer + 5))) << 40) + \
(((UINT64)(*((_s)->pointer + 6))) << 48) + \
(((UINT64)(*((_s)->pointer + 7))) << 56)); \
if (_p) \ if (_p) \
Stream_Seek(_s, sizeof(_t)); \ Stream_Seek(_s, sizeof(_t)); \
} while (0) } while (0)
@ -155,10 +157,12 @@ extern "C"
{ \ { \
WINPR_ASSERT(_s); \ WINPR_ASSERT(_s); \
WINPR_ASSERT(Stream_GetRemainingLength(_s) >= 8); \ WINPR_ASSERT(Stream_GetRemainingLength(_s) >= 8); \
(_v) = (_t)( \ (_v) = \
(((UINT64)(*((_s)->pointer))) << 56) + (((UINT64)(*((_s)->pointer + 1))) << 48) + \ (_t)((((UINT64)(*((_s)->pointer))) << 56) + (((UINT64)(*((_s)->pointer + 1))) << 48) + \
(((UINT64)(*((_s)->pointer + 2))) << 40) + (((UINT64)(*((_s)->pointer + 3))) << 32) + \ (((UINT64)(*((_s)->pointer + 2))) << 40) + \
(((UINT64)(*((_s)->pointer + 4))) << 24) + (((UINT64)(*((_s)->pointer + 5))) << 16) + \ (((UINT64)(*((_s)->pointer + 3))) << 32) + \
(((UINT64)(*((_s)->pointer + 4))) << 24) + \
(((UINT64)(*((_s)->pointer + 5))) << 16) + \
(((UINT64)(*((_s)->pointer + 6))) << 8) + (((UINT64)(*((_s)->pointer + 7))))); \ (((UINT64)(*((_s)->pointer + 6))) << 8) + (((UINT64)(*((_s)->pointer + 7))))); \
if (_p) \ if (_p) \
Stream_Seek(_s, sizeof(_t)); \ Stream_Seek(_s, sizeof(_t)); \
@ -215,7 +219,7 @@ extern "C"
#define Stream_Peek_UINT64_BE(_s, _v) _stream_read_n64_be(UINT64, _s, _v, FALSE) #define Stream_Peek_UINT64_BE(_s, _v) _stream_read_n64_be(UINT64, _s, _v, FALSE)
#define Stream_Peek_INT64_BE(_s, _v) _stream_read_n64_be(INT64, _s, _v, FALSE) #define Stream_Peek_INT64_BE(_s, _v) _stream_read_n64_be(INT64, _s, _v, FALSE)
static INLINE void Stream_Peek(wStream* _s, void* _b, size_t _n) static INLINE void Stream_Peek(const wStream* _s, void* _b, size_t _n)
{ {
WINPR_ASSERT(_s); WINPR_ASSERT(_s);
WINPR_ASSERT(_b || (_n == 0)); WINPR_ASSERT(_b || (_n == 0));
@ -368,7 +372,7 @@ extern "C"
void Stream_SetCapacity(wStream* _s, size_t capacity)); void Stream_SetCapacity(wStream* _s, size_t capacity));
#endif #endif
static INLINE size_t Stream_Length(wStream* _s) static INLINE size_t Stream_Length(const wStream* _s)
{ {
WINPR_ASSERT(_s); WINPR_ASSERT(_s);
return _s->length; return _s->length;
@ -377,7 +381,7 @@ extern "C"
#define Stream_GetLength(_s, _l) _l = Stream_Length(_s) #define Stream_GetLength(_s, _l) _l = Stream_Length(_s)
WINPR_API BOOL Stream_SetLength(wStream* _s, size_t _l); WINPR_API BOOL Stream_SetLength(wStream* _s, size_t _l);
static INLINE size_t Stream_Capacity(wStream* _s) static INLINE size_t Stream_Capacity(const wStream* _s)
{ {
WINPR_ASSERT(_s); WINPR_ASSERT(_s);
return _s->capacity; return _s->capacity;
@ -385,7 +389,7 @@ extern "C"
#define Stream_GetCapacity(_s, _c) _c = Stream_Capacity(_s); #define Stream_GetCapacity(_s, _c) _c = Stream_Capacity(_s);
static INLINE size_t Stream_GetPosition(wStream* _s) static INLINE size_t Stream_GetPosition(const wStream* _s)
{ {
WINPR_ASSERT(_s); WINPR_ASSERT(_s);
WINPR_ASSERT(_s->buffer <= _s->pointer); WINPR_ASSERT(_s->buffer <= _s->pointer);

4
winpr/libwinpr/utils/stream.c
View File

@ -256,7 +256,7 @@ void Stream_SetCapacity(wStream* _s, size_t _c)
#endif #endif
size_t Stream_GetRemainingCapacity(wStream* _s) size_t Stream_GetRemainingCapacity(const wStream* _s)
{ {
size_t cur; size_t cur;
WINPR_ASSERT(_s); WINPR_ASSERT(_s);
@ -272,7 +272,7 @@ size_t Stream_GetRemainingCapacity(wStream* _s)
return (_s->capacity - cur); return (_s->capacity - cur);
} }
size_t Stream_GetRemainingLength(wStream* _s) size_t Stream_GetRemainingLength(const wStream* _s)
{ {
size_t cur; size_t cur;
WINPR_ASSERT(_s); WINPR_ASSERT(_s);