RFR: 8295017: Remove Windows specific workaround in JLI_Snprintf [v5]
Sergey Bylokhov
serb at openjdk.org
Wed Jul 26 04:34:57 UTC 2023
On Tue, 25 Jul 2023 23:51:34 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
>> Julian Waters has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
>>
>> - Merge branch 'openjdk:master' into patch-1
>> - Merge branch 'openjdk:master' into patch-1
>> - Comment documenting change isn't required
>> - Merge branch 'openjdk:master' into patch-1
>> - Comment formatting
>> - Remove Windows specific JLI_Snprintf implementation
>> - Remove Windows JLI_Snprintf definition
>
> Thank you!
>
>>If processing string specifier s, S, or Z, format specification processing stops, a NULL is placed at the beginning of the buffer.
>
> I hope this is not an MS extension/implementation detail since I did not find this in any other places.
>@mrserb this change was to a Windows specific file.
That change removed the windows specific version of the JLI_Snprintf, and now we use
`#define JLI_Snprintf snprintf` on all platforms. And my question was about that "cross-platform" `snprintf`. As linked in the comment above on Windows it adds the null at the start of the buffer in case of error when a negative value is returned. But is that specified by the c99?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/10625#issuecomment-1650963715
More information about the core-libs-dev
mailing list