[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] [generics] RFC: Genericize return type in PrinterStateR

From: Wolfgang Baer
Subject: Re: [cp-patches] [generics] RFC: Genericize return type in PrinterStateReasons
Date: Sun, 15 Jan 2006 14:45:46 +0100
User-agent: Debian Thunderbird 1.0.7 (X11/20051017)


Andrew John Hughes wrote:
On Sat, 2006-01-14 at 11:54 +0100, Wolfgang Baer wrote:

The patch looks fine to me; I think I already did some similiar edits on
this particular file when merging, so please make sure your patch is
against the latest version before committing.


Generally, I think these kind of edits are okay, but I'd save the
majority of it, where possible, to after the generics branch is
re-merged.  There are cases where it has to be done, and I also don't
see any problem with fixing the bits that are part of the API (this is
especially important for those who worship JAPI).  Additionally, it
could possibly catch the odd bug... though I've not seen it yet.

So I only committing the part of the API and just ignore the eclipse
warning for the stuff inside the method.

2006-01-15  Wolfgang Baer  <address@hidden>

        * javax/print/attribute/standard/
        (printerStateReasonSet): Genericize the return type.


RCS file: 
retrieving revision
diff -u -r1.2.2.5
---    10 Jan 2006 15:59:42 -0000
+++    15 Jan 2006 13:39:47 -0000
@@ -139,7 +139,7 @@
    * @param severity the severity level for the constructed set.
    * @return The set of printer state reasons.
-  public Set printerStateReasonSet(Severity severity)
+  public Set<PrinterStateReason> printerStateReasonSet(Severity severity)
     if (severity == null)
       throw new NullPointerException("severity is null");

reply via email to

[Prev in Thread] Current Thread [Next in Thread]