comment on public ARCFileReader constructor is confusing

[banshee]

On line 102 of commoncrawl/src/main/java/org/commoncrawl/util/shared/ARCFileReader.java, there's a comment that says the constructor is private (it's actually public), and refers to the "factory method above" even though it's the first method in the file.

  /** 
   * constructor is now private. use the factory method above to construct a reader 
   * @param source
   * @throws IOException
   */
  public ARCFileReader(final InputStream source)throws IOException {
    super(new CustomPushbackInputStream(new CountingInputStream(source),
        _blockSize), new Inflater(true), _blockSize);
    readARCHeader();
  }

[sebastian-nagel]

Thanks, I'll have a look at it. But just for clarification: this library is used to access the Common Crawl data from 2012 or earlier. Recent data uses a different format (WARC instead of ARC).

[banshee]

Oh, that's a great reason to just close as won't-fix. I had just started poking around the code and I forgot that distinction.