[jboss-cvs] JBossAS SVN: r64480 - projects/microcontainer/trunk/deployers-impl/src/main/org/jboss/deployers/plugins/classloading.

[Adrian]

??? The module can't be null otherwise it wouldn't work at all!

See the resolve method.

On Tue, 2007-08-07 at 06:55 -0400, jboss-cvs-commits at lists.jboss.org
wrote:
> Author: alesj
> Date: 2007-08-07 06:55:58 -0400 (Tue, 07 Aug 2007)
> New Revision: 64480
> 
> Modified:
>    projects/microcontainer/trunk/deployers-impl/src/main/org/jboss/deployers/plugins/classloading/RequirementDependencyItem.java
> Log:
> Typo - check for null module before getName() call.
> 
> Modified: projects/microcontainer/trunk/deployers-impl/src/main/org/jboss/deployers/plugins/classloading/RequirementDependencyItem.java
> ===================================================================
> --- projects/microcontainer/trunk/deployers-impl/src/main/org/jboss/deployers/plugins/classloading/RequirementDependencyItem.java	2007-08-07 10:09:55 UTC (rev 64479)
> +++ projects/microcontainer/trunk/deployers-impl/src/main/org/jboss/deployers/plugins/classloading/RequirementDependencyItem.java	2007-08-07 10:55:58 UTC (rev 64480)
> @@ -54,7 +54,7 @@
>      */
>     public RequirementDependencyItem(Module module, Requirement requirement)
>     {
> -      super(module.getName(), null, CLASSLOADER, CLASSLOADER);
> +      super(module != null ? module.getName() : null, null, CLASSLOADER, CLASSLOADER);
>        if (module == null)
>           throw new IllegalArgumentException("Null module");
>        if (requirement == null)
> 
> _______________________________________________
> jboss-cvs-commits mailing list
> jboss-cvs-commits at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-cvs-commits
-- 
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
Adrian Brock
Chief Scientist
JBoss, a division of Red Hat
xxxxxxxxxxxxxxxxxxxxxxxxxxxx