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.


Reply via email to