[8] Review request for 7149062: [macosx] dock menu don't show available frames

Anthony Petrov anthony.petrov at oracle.com
Fri May 4 09:09:38 PDT 2012


Thanks for the review.

--
best regards,
Anthony

On 5/4/2012 8:07 PM, Scott Kovatch wrote:
> On May 4, 2012, at 4:43 AM, Anthony Petrov wrote:
> 
>> I've investigated that, but it looks like a category (or an extension) requires a class name to add methods to. It's impossible to define an anonymous category and then apply it to two different classes. However, for the sake of this fix we do need to have two distinct classes: one inheriting from NSWindow, and another - from NSPanel. I can't find anything Obj-C-wise that could help eliminate the code replication here. Well, we could extract it into a separate file and #include it within the corresponding @implementation sections. But given the really small size of the replicated code this would look a bit weird in my opinion.
> 
> No problem -- it was more a random thought than a concrete suggestion. I doubt we will be changing much in either file over time.
> 
>> So I re-generated the webrev w/o the UTILITY thing mentioned above. The final webrev is at:
>>
>> http://cr.openjdk.java.net/~anthony/8-26-windowListInDockMenu-7149062.1/
>>
>> Looks OK to push that?
> 
> Looks good to me.
> 
> -- Scott K.
> 
> 


More information about the macosx-port-dev mailing list