| Subject: | Poor choice of local package names |
PObject/Template.pm has local packages named INTEGER, CHAR, MD5, etc. that
seem to be problematic. Good practice would advise to put all packages of a
distribution into a common namespace. The packages here populate top level
namespaces not registered for this purpose, outside the main namespace of the
distribution and much more generic than the scope of the containing module
would suggest. Moreover, on platforms with case-insensitive file systems they
may clash with existing modules like integer.pm.
To resolve this issue, those packages could be renamed
Class::PObject::Template::Integer etc.
-Martin