You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oro-dev@jakarta.apache.org by df...@apache.org on 2004/02/18 04:11:34 UTC

cvs commit: jakarta-oro/src/java/org/apache/oro/text/perl Perl5Util.java

dfs         2004/02/17 19:11:34

  Modified:    src/java/org/apache/oro/text/perl Perl5Util.java
  Log:
  Fixed issue report 26892
    http://nagoya.apache.org/bugzilla/show_bug.cgi?id=26892
  reported by sebb@apache.org:
  
  >The Javadoc has some bugs:
  >
  >http://jakarta.apache.org/oro/api/org/apache/oro/text/perl/Perl5Util.html
  >has an example which includes the following:
  >
  > while((line = input.readLine()) != null) {
  >     if(util.match("/HREF=\"description1.html\"/")) {
  >
  >However, match() requires 2 parameters, it should read:
  >
  >     if(util.match("/HREF=\"description1.html\"/",line) {
  
  Revision  Changes    Path
  1.21      +2 -2      jakarta-oro/src/java/org/apache/oro/text/perl/Perl5Util.java
  
  Index: Perl5Util.java
  ===================================================================
  RCS file: /home/cvs/jakarta-oro/src/java/org/apache/oro/text/perl/Perl5Util.java,v
  retrieving revision 1.20
  retrieving revision 1.21
  diff -u -r1.20 -r1.21
  --- Perl5Util.java	13 Feb 2004 22:01:59 -0000	1.20
  +++ Perl5Util.java	18 Feb 2004 03:11:34 -0000	1.21
  @@ -74,7 +74,7 @@
    * while((line = input.readLine()) != null) {
    *     // First find the line with the string we want to substitute because
    *     // it is cheaper than blindly substituting each line.
  - *     if(util.match("/HREF=\"description1.html\"/")) {
  + *     if(util.match("/HREF=\"description1.html\"/", line)) {
    *        line = util.substitute("s/description1\\.html/about1.html/", line);
    *     }
    *    output.println(line);
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: oro-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: oro-dev-help@jakarta.apache.org