commons-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hendrik Maryns <hendrik.mar...@uni-tuebingen.de>
Subject [Collections] Javadoc in DefaultedMap very inconsistent
Date Thu, 06 Jul 2006 08:04:57 GMT
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi all,

The javadocs in DefaultedMap is very inconsistent.  It speaks of factory
when a transformer is meant, it speaks of transformer when value is
meant and that sort of stuff.

Besides, it would be much cleaner to declare ‘value’ of type
Transformer, and create a transformer every time, except when one is
passed (the instanceof checks are wrong anyway), (I’d rename it to
transformer, too).

Even nicer would be to split up the constructor(s) into one that takes a
value, and one that takes a Transformer, no instanceof checks needed,
much cleaner code, generification is almost trivial :-)

I attached what I made of it, removing the generic markers gives a
perfectly legal 1.4 file.

Cheers, H.
- --
Hendrik Maryns
Herrenberger Straße 40
D-72070 Tübingen
+49707143783    (Deutschland fest)
+491753353494    (Deutschland Handy)
+32485470741    (België)

=================
www.lieverleven.be    (waarom zou ik hier toch naar verwijzen...)
http://aouw.org        De kunst van het stedelijke oorlogvoeren
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)

iD8DBQFErMQpBGFP0CTku6MRAnheAJ9CHhbUlXQ/xhdlQdFZLmAsSsi3rgCfdAYp
G2WkCErp39PI3mibC3Nch6E=
=YwE5
-----END PGP SIGNATURE-----

Mime
View raw message