On Thu, 5 Nov 2020 17:52:53 GMT, Phil Race wrote:
>> @mrserb Good catch! Yes indeed, it looks like `range-loop-analysis` needs to
>> be disabled for clang for Xcode 12 to not complain.
>
> I added that to the disabled warnings .. now I suppose the actions will
> re-run but I don't see any sign
On Thu, 5 Nov 2020 13:36:18 GMT, Magnus Ihse Bursie wrote:
>> Looks like the build on macOS via GitHub will be broken by this fix?
>
> @mrserb Good catch! Yes indeed, it looks like `range-loop-analysis` needs to
> be disabled for clang for Xcode 12 to not complain.
I added that to the disabled
> This upgrades JDK to import the current 2.7.2 version of harfbuzz - an
> OpenType text shaping library
>
> https://bugs.openjdk.java.net/browse/JDK-8247872
>
> This has passed building and headless and headful automated tests on all
> platforms.
Phil Race has updated the pull request increme
On Thu, 5 Nov 2020 11:24:28 GMT, Sergey Bylokhov wrote:
>>> I'm just a bit curious about the added, empty,
>>> `src/java.desktop/share/native/libharfbuzz/abc.txt`...
>>>
>>> If it really is in upstream source, I'm not saying you should remove it. It
>>> just looks very odd. It's not a merge ar
On Thu, 5 Nov 2020 09:06:39 GMT, Sergey Bylokhov wrote:
> The test leaves two temporary files due to unclosed file streams.
This pull request has now been integrated.
Changeset: d3c43c28
Author:Sergey Bylokhov
URL: https://git.openjdk.java.net/jdk/commit/d3c43c28
Stats: 19 lines
On Mon, 2 Nov 2020 16:23:19 GMT, Phil Race wrote:
>> I'm just a bit curious about the added, empty,
>> `src/java.desktop/share/native/libharfbuzz/abc.txt`...
>>
>> If it really is in upstream source, I'm not saying you should remove it. It
>> just looks very odd. It's not a merge artifact?
>>
On Thu, 5 Nov 2020 09:06:39 GMT, Sergey Bylokhov wrote:
> The test leaves two temporary files due to unclosed file streams.
Marked as reviewed by jdv (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/1074
The test leaves two temporary files due to unclosed file streams.
-
Commit messages:
- Update StreamFlush.java
Changes: https://git.openjdk.java.net/jdk/pull/1074/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1074&range=00
Issue: https://bugs.openjdk.java.net/browse
