BCEL Bug.

[da...@forgeahead.hu]

As i am not used to using diff and its just a oneline bug i will just
copy paste the code.


org.apache.util.Classpath

line 192 - 207

  /**
   * @param name fully qualified class name, e.g. java.lang.String
   * @return input stream for class
   */
  public InputStream getInputStream(String name) throws IOException {
    return getInputStream(name, ".class");
  }

  /**
   * Return stream for class or resource on CLASSPATH.
   *
   * @param name fully qualified file name, e.g. java/lang/String
   * @param suffix file name ends with suff, e.g. .java
   * @return input stream for file on class path
   */
  public InputStream getInputStream(String name, String suffix)


The problem is that getInputStream(String name) claims to accept
"java.lang.String" class name and then it uses the
getInputStream(String name, String suffix) method that wants "java/lang/String".

So the fix would be

return getInputStream(name.replace('.','/'), ".class");

on line 197.

BR Bjorn Blowqvist

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

Re: BCEL Bug.

[bj...@forgeahead.hu]

Great to hear that its fixed =)

Next time i will use diff.

On Tue, 17 May 2005, Torsten Curdt wrote:

> > As i am not used to using diff and its just a oneline bug i will just
> > copy paste the code.
>
> Thanks for the "patch" ...but please get
> used to usinng diff ;) The line numbers
> were not correct anymore ...makes it a
> bit more work to find the right spot.
>
> cheers
> --
> Torsten
>

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

Re: BCEL Bug.

[Torsten Curdt <tc...@apache.org>]

> As i am not used to using diff and its just a oneline bug i will just
> copy paste the code.

Thanks for the "patch" ...but please get
used to usinng diff ;) The line numbers
were not correct anymore ...makes it a
bit more work to find the right spot.

cheers
--
Torsten