It certainly could be, but it isn't required.  What if the factory
method used a constructor that took arguments?  We could go in
circles on this one forever...it all depends on how the author
writes his class.

Part of the answer to this would be to figure out what the goal of
marking items as unused should be.  If the goal is to have no
unused items marked in a project (my initial thought), then you
must special case this as it is a common occurance.  If the goal
of marking items unused is just a reminder or a 'oh, by the way'
helper, then leave it marked.

-- Rick

> -----Original Message-----
> From: Jason Boehle [mailto:[EMAIL PROTECTED]]
> Sent: Thursday, February 07, 2002 8:21 AM
> To: '[EMAIL PROTECTED]'
> Subject: RE: [Eap-list] Bugs with Ariadna's unused members/local
> variable feature.
> 
> 
> But shouldn't it then be used within that class, like in a 
> factory method?
> 
> Jason Boehle
> [EMAIL PROTECTED]
> 
> 
> -----Original Message-----
> From: Vestal, Rick [mailto:[EMAIL PROTECTED]] 
> Sent: Thursday, February 07, 2002 6:57 AM
> To: '[EMAIL PROTECTED]'
> Subject: RE: [Eap-list] Bugs with Ariadna's unused 
> members/local variable
> feature.
> 
> 
> I think only a private constructor that takes no arguments should be
> made the special case (i.e. don't mark it as unused).  Lots of people
> do this to make sure that you cannot create an instance of this class
> from outside the class itself.  So, the fact that it is 
> unused is true, but
> it is intentional and shouldn't be marked as unused.
> 
> -- Rick
> -----Original Message-----
> From: Valentin Kipiatkov [mailto:[EMAIL PROTECTED]]
> Sent: Thursday, February 07, 2002 4:15 AM
> To: [EMAIL PROTECTED]
> Subject: Re: [Eap-list] Bugs with Ariadna's unused 
> members/local variable
> feature.
> 
> 
> > Private constructors that are not used should not be marked 
> as unused.
> 
> Why?
> 
> _______________________________________________
> Eap-list mailing list
> [EMAIL PROTECTED]
> http://www.intellij.com/mailman/listinfo/eap-list
> 

_______________________________________________
Eap-list mailing list
[EMAIL PROTECTED]
http://www.intellij.com/mailman/listinfo/eap-list

Reply via email to