struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Martin Cooper" <mart...@apache.org>
Subject Re: svn commit: r551067 - /struts/struts1/trunk/taglib/src/main/java/org/apache/struts/taglib/html/XhtmlTag.java
Date Thu, 28 Jun 2007 18:53:05 GMT
The "best" part is debatable - it's certainly not one that I favour. ;-)
IMHO, it's redundant and clutters the code.

--
Martin Cooper


On 6/28/07, Paul Benedict <pbenedict@apache.org> wrote:
>
> That's not why I added it. I added it out of best practice.
>
> On 6/28/07, Martin Cooper <martinc@apache.org> wrote:
> >
> > On 6/26/07, pbenedict@apache.org <pbenedict@apache.org> wrote:
> > >
> > > Author: pbenedict
> > > Date: Tue Jun 26 23:05:09 2007
> > > New Revision: 551067
> > >
> > > URL: http://svn.apache.org/viewvc?view=rev&rev=551067
> > > Log:
> > > STR-1175: Add instance qualification
> >
> >
> > Why? There doesn't appear to be any local variable for this one to be
> > confused with.
> >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message