mmm ? I don't really understand what you imply here.
More simply why isn't this "$Header$" (okay this is compiled directly this
way) ? :)

If the original author can enlighten us about this part that would be nice,
I was ready to refactor all this in an ArchiveFormat.format() and
ArchiveFormat.clean() and move to Oro instead of gnu.regex but I'm kind of
lost about the substitution here. I'm 99% sure it is wrong but need feedback
to be sure at 100%.

> -----Original Message-----
> From: Jon Scott Stevens [mailto:[EMAIL PROTECTED]]
> 
> Why isn't this ' instead of ":
> 
> '$' + "Header" + '$'
> 
> ??????
>
> on 2/24/02 5:52 AM, "[EMAIL PROTECTED]" 
> <[EMAIL PROTECTED]> wrote:
> 
> > sbailliez    02/02/24 05:52:28
> > 
> > Modified:    src/java/org/apache/maven/jrcs/rcs Archive.java
> > Log:
> > Fixed typo about 'RCSFile' instead of 'RCSfile'.
> > I don't understand how this part works about the
> > substitution since the sequential substitutes in the loop
> > actually never use the result of the preceding substitute call.
> > 
> > Revision  Changes    Path
> > 1.6       +1 -1  
> > 
> jakarta-turbine-maven/src/java/org/apache/maven/jrcs/rcs/Archive.java
> > 
> > Index: Archive.java
> > ===================================================================
> > RCS file: 
> > 
> /home/cvs/jakarta-turbine-maven/src/java/org/apache/maven/jrcs
> /rcs/Archive.jav
> > a,v
> > retrieving revision 1.5
> > retrieving revision 1.6
> > diff -u -r1.5 -r1.6
> > --- Archive.java    23 Feb 2002 13:48:35 -0000    1.5
> > +++ Archive.java    24 Feb 2002 13:52:28 -0000    1.6
> > @@ -832,7 +832,7 @@
> >              result[i] = Id_re.substituteAll(text[i], "$" + 
> "Id" + "$");
> >              result[i] = Header_re.substituteAll(text[i], 
> "$" + "Header" +
> > "$");
> >              result[i] = Source_re.substituteAll(text[i], 
> "$" + "Source" +
> > "$");
> > -            result[i] = RCSfile_re.substituteAll(text[i], 
> "$" + "RCSFile" +
> > "$");
> > +            result[i] = RCSfile_re.substituteAll(text[i], 
> "$" + "RCSfile" +
> > "$");
> >              result[i] = Revision_re.substituteAll(text[i], 
> "$" + "Revision" +
> > "$");
> >              result[i] = Date_re.substituteAll(text[i], "$" 
> + "Date" + "$");
> >              result[i] = Author_re.substituteAll(text[i], 
> "$" + "Author" +
> > "$");
> > 
> > 
> > 
> > 
> > --
> > To unsubscribe, e-mail:   
<mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>

-- 
Standard rules apply: Ask any questions, and you get the job. ;-)


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

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

Reply via email to