<AWT Dev> [9] Review Request: 8141553 JDK fails to build with Xcode 7 on 10.11

[Avik Niyogi]

Here is my two cents:
As of Xcode 7, Rich Text Font Document attributes have to be mentioned explicitly and not with a nil.
I can see that to mitigate the warning an empty array is placed there. The system is able to still process RTF document with this.
Ideally, the parameter to be passed for the document attribute as per my understanding is @{NSDocumentTypeDocumentAttribute: NSRTFTextDocumentType}
at line numbers 627 and 684 AWTView.m
Please refer to the below link for further clarification regarding values to be passed (like NULL instead of nil):
https://developer.apple.com/library/prerelease/mac/documentation/Cocoa/Reference/ApplicationKit/Classes/NSAttributedString_AppKitAdditions/#//apple_ref/occ/instm/NSAttributedString/initWithRTFD:documentAttributes: <https://developer.apple.com/library/prerelease/mac/documentation/Cocoa/Reference/ApplicationKit/Classes/NSAttributedString_AppKitAdditions/#//apple_ref/occ/instm/NSAttributedString/initWithRTFD:documentAttributes:>

performFileOperation at line 224 is deprecated in OSX10.11. 
recycleURLs:completionHandler: should be used instead as far as it is possible (until a good substitute for performFileOperation is available)
https://developer.apple.com/library/mac/releasenotes/AppKit/RN-AppKit/ <https://developer.apple.com/library/mac/releasenotes/AppKit/RN-AppKit/>


With Regards,
Avik Niyogi

> On 10-Feb-2016, at 11:46 pm, Chris Bensen <chris.bensen at oracle.com> wrote:
> 
>> On Feb 10, 2016, at 9:16 AM, David DeHaven <david.dehaven at oracle.com> wrote:
>> 
>> 
>>> Please review one more iteration of the fix of warnings in awt on osx.
>>> The fix was contributed by Chris Bensen.
>>> 
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8141553
>>> Webrev can be found at: http://cr.openjdk.java.net/~serb/8141553/webrev.00
>> 
>> Thanks for doing this! I'm not a (R)eviewer, but I've been using these changes for some time now with no ill effects.
>> 
>> -DrD-
> 
> +1
> 
> Chris

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20160211/7a6054f8/attachment.html>