Can you shut the f Le mer. 9 août 2023, 10:10, Thomas Stuefe <stu...@openjdk.org> a écrit :
> On Wed, 9 Aug 2023 06:53:49 GMT, David Holmes <dhol...@openjdk.org> wrote: > > >> I wrote this code ages ago. I'm not sure what's weird or suspicious > about it, though. The comment at the file's beginning explains this code's > motivation. > >> > >> The buffer was never thought to be used for something different than > HANDLEs or characters, where the assignment of integer literals work. I > often use char constants for sentinels as debugging aid. As for `'\0'`, > that indicates to the casual code reader that this is a termination of a > string, better than had I used a plain 0. > > > > Because there is nothing to state what T may be, I found assigning > character literals to be odd. If T is char and the buffer is meant to be a > C string then it makes more sense. But for non-char T it just raised > questions for me. > > I see that. We can remove the sentinel stuff, which leaves us with the > zero termination. Arguably, this could be done by sub classes that derive > from the char instantiation. > > ------------- > > PR Review Comment: > https://git.openjdk.org/jdk/pull/15096#discussion_r1288083649 >