Re: Extending the Description class

Mark,

I'd like to understand a little better what you're doing.  Are you
fine-tuning the performance of a HashMap with Descriptions as keys?

   David Saff

On Tue, Oct 14, 2008 at 3:02 AM, Mark Stobbe <[email protected]> wrote:
> Hello JUnit developer team,
>
> I have a question, or rather a request: could you make the constructor (and
> possibly the fields) of org.junit.runner.Description "protected" instead of
> "private". In this way it is impossible to normally subclass Description
> with a custom implementation.
>
> Why would you like to subclass Description? Well, the latest implementation
> of Description has a rather difficult hashCode implementation, namely:
> hashCode = getDisplayName().hashCode(). This poses a problem (or not, but in
> my case it does) when you create multiple instances of the Description class
> with the same name.
>
> Hopefully you are willing to incorporate this minor modification, it would
> surely make my life easier.
>
> Big thanks,
> Mark Stobbe
> ________________________________
> Plan je evenement, nodig mensen uit en deel je foto's met Windows Live
> Events
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great
> prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Junit-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/junit-devel
>
>

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/