On Fri, 12 Nov 2004 10:40:59 +0700, Kevin Jackson <[EMAIL PROTECTED]> wrote: > This looks like dead code as all the functionality is in AbstractCvsTask. > > Would it be possible to prune this from the current version? > > Kev >
Its retained in case someone, somewhere, had subclassed the task, or created a cvs task and cast it to tasks.Cvs. In OSS code there is no real privacy of implementation, and we have to try and maintain the appearance of stability in the moderately public areas. That doesnt mean that we cant change things, but we have to assume that people use our tasks programatically as well as in XML and so cannot -remove existing tasks -remove existing setter methods. If you want to change the type of a setter (e.g String -> Integer), you need to retain the old one *after* the new one. -Steve --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
