diff options
| author | Rémi Verschelde <rverschelde@gmail.com> | 2019-06-15 12:11:30 +0200 |
|---|---|---|
| committer | Rémi Verschelde <rverschelde@gmail.com> | 2019-06-15 14:40:45 +0200 |
| commit | 68735d2a886bc2cf6b0a4300d1aa5ece952929ed (patch) | |
| tree | 2941482adee4afc860bca3fb9968f40114249d2d /modules/webrtc | |
| parent | 0a3c21d999559617cc9cdfe261d631e6d1267374 (diff) | |
Fix compilation warnings in JS and Windows builds
Warnings raised by Emscripten 1.38.0 and MinGW64 5.0.4 / GCC 8.3.0.
JS can now build with `werror=yes warnings=extra`.
MinGW64 still has a few warnings to resolve with `warnings=extra`,
and only one with `warnings=all`.
Part of #29033 and #29801.
Diffstat (limited to 'modules/webrtc')
| -rw-r--r-- | modules/webrtc/webrtc_data_channel_js.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/modules/webrtc/webrtc_data_channel_js.cpp b/modules/webrtc/webrtc_data_channel_js.cpp index ce2fada634..069918cc9c 100644 --- a/modules/webrtc/webrtc_data_channel_js.cpp +++ b/modules/webrtc/webrtc_data_channel_js.cpp @@ -68,7 +68,7 @@ void WebRTCDataChannelJS::_on_error() { } void WebRTCDataChannelJS::_on_message(uint8_t *p_data, uint32_t p_size, bool p_is_string) { - if (in_buffer.space_left() < p_size + 5) { + if (in_buffer.space_left() < (int)(p_size + 5)) { ERR_EXPLAIN("Buffer full! Dropping data"); ERR_FAIL(); } |