You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ak...@hyperreal.org on 1999/11/13 01:16:03 UTC

cvs commit: jakarta-tomcat/src/share/org/apache/jasper/runtime BodyContentImpl.java

akv         99/11/12 16:16:03

  Modified:    src/share/org/apache/jasper/runtime BodyContentImpl.java
  Log:
  BodyContentImpl swallows the last character of a tag body. Thanks, Glenn.
  Submitted by: Glenn Atter <ga...@oxmol.co.uk>
  Reviewed by: Anil V <ak...@eng.sun.com>
  
  Revision  Changes    Path
  1.5       +3 -3      jakarta-tomcat/src/share/org/apache/jasper/runtime/BodyContentImpl.java
  
  Index: BodyContentImpl.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat/src/share/org/apache/jasper/runtime/BodyContentImpl.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- BodyContentImpl.java	1999/11/08 03:14:42	1.4
  +++ BodyContentImpl.java	1999/11/13 00:16:02	1.5
  @@ -549,7 +549,7 @@
        * @returns the value of this BodyJspWriter as a Reader
        */
       public Reader getReader() {
  -        return new CharArrayReader (cb, 0, nextChar-1);
  +        return new CharArrayReader (cb, 0, nextChar);
       }
   
       /**
  @@ -560,7 +560,7 @@
        * @returns the value of the BodyJspWriter as a String
        */
       public String getString() {
  -        return new String(cb, 0, nextChar-1);
  +        return new String(cb, 0, nextChar);
       }
   	
       /**
  @@ -572,7 +572,7 @@
        * this body evaluation
        */
       public void writeOut(Writer out) throws IOException {
  -        out.write(cb, 0, nextChar-1);
  +        out.write(cb, 0, nextChar);
   	//Flush not called as the writer passed could be a BodyContent and
   	//it doesn't allow to flush.
       }