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 <[email protected]> wrote:
>
>> On Feb 10, 2016, at 9:16 AM, David DeHaven <[email protected]> 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
