On Fri, 11 Aug 2023 20:15:59 GMT, Daniel Jeliński wrote:
>> Please review this patch that fixes the scope of 2 local variables to make
>> sure they are in scope when they are used.
>>
>> The patch additionally fixes one `c++11-narrowing` Clang warning by
>> explicitly casting string size (`siz
On Fri, 11 Aug 2023 20:15:59 GMT, Daniel Jeliński wrote:
>> Please review this patch that fixes the scope of 2 local variables to make
>> sure they are in scope when they are used.
>>
>> The patch additionally fixes one `c++11-narrowing` Clang warning by
>> explicitly casting string size (`siz
> Please review this patch that fixes the scope of 2 local variables to make
> sure they are in scope when they are used.
>
> The patch additionally fixes one `c++11-narrowing` Clang warning by
> explicitly casting string size (`size_t`) to `DWORD`.
Daniel Jeliński has updated the pull request
On Fri, 11 Aug 2023 16:53:41 GMT, Mark Powers wrote:
>> Please review this patch that fixes the scope of 2 local variables to make
>> sure they are in scope when they are used.
>>
>> The patch additionally fixes one `c++11-narrowing` Clang warning by
>> explicitly casting string size (`size_t`
On Fri, 11 Aug 2023 06:36:42 GMT, Daniel Jeliński wrote:
> Please review this patch that fixes the scope of 2 local variables to make
> sure they are in scope when they are used.
>
> The patch additionally fixes one `c++11-narrowing` Clang warning by
> explicitly casting string size (`size_t`)
Please review this patch that fixes the scope of 2 local variables to make sure
they are in scope when they are used.
The patch additionally fixes one `c++11-narrowing` Clang warning by explicitly
casting string size (`size_t`) to `DWORD`.
-
Commit messages:
- Fix variable scope
