[EMAIL PROTECTED] writes:

>  -    public static void setTurbineServletConfig(ServletConfig s)
>  +    public static void setTurbineServletConfig(ServletConfig config)
>       {
>  -        servletConfig = s;
>  +        servletConfig = config;

Can we use  "this.servletConfig = servletConfig"?

>  -        servletContext = s;
>  +        servletContext = context;

same.

>  -        String[] names = data.getSession().getValueNames();
>  -        if (names != null)
>  +        for( Enumeration enum = data.getSession().getAttributeNames();
>  +                enum.hasMoreElements(); )

I'm not sure but this might have been a bug workaround for some buggy
servlet containers, which return null for
getValueNames()/getAttributeNames() if none are set. You might want to
check. Not sure about this, though.

        Regards
                Henning

-- 
Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
[EMAIL PROTECTED]        +49 9131 50 654 0   http://www.intermeta.de/

Java, perl, Solaris, Linux, xSP Consulting, Web Services 
freelance consultant -- Jakarta Turbine Development  -- hero for hire

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to