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.
