RFR: 8343802: Prevent NULL usage backsliding [v2]
Chen Liang
liach at openjdk.org
Thu Feb 6 05:08:10 UTC 2025
On Wed, 5 Feb 2025 19:05:24 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Nizar Benalla has updated the pull request incrementally with one additional commit since the last revision:
>>
>> update based on feedback
>
> test/hotspot/jtreg/sources/TestNoNULL.java line 103:
>
>> 101: public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) {
>> 102: if (isIncluded(file, excludedFiles, excludeExtensions)) {
>> 103: files.add(file);
>
> Why collect files to be checked, rather than just checking them here?
I agree with @kimbarrett; there is no point creating buffer collections that consume more resources with no real gains.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23466#discussion_r1944101310
More information about the hotspot-dev
mailing list