Support reading GZIPInputStream log files about gcviewer

[hbswn]

Also works over HTTP, which is very convenient, if you have Apache transparently gzip the files.

[chewiebug]

Hi Hans,

This looks like an interesting feature! Do you know that you could submit a patch as a pull request? Alternatively you can tell me the email address that is associated to your github account so I can author a commit in your name (write it here or send me an email to [email protected]).

Please let me know how you would like to proceed.

Regards, Jörg

[hbswn]

New code is on https://github.com/hbswn/GCViewer

and I've sent a Pull Request ... but I do not see it.

[chewiebug]

Thank you for the pull request! I'll probably pull it in within the next two weeks.

I am not sure if you can see it in your repository. It appears in my list of pull requests and as another issue.

...yes, a testcase for gzip'd data would be nice!

Regards, Jörg

[hbswn]

I've added:
TestDataReaderFactory.testReadUShortAndReset()
TestDataReaderFactory.testGetDataReaderJDK6GZipped()

... and made DataReaderFactory.readUShortAndReset() protected to make it accessible for the test class.

Tests run: 177, Failures: 0, Errors: 0, Skipped: 0

[chewiebug]

cool, thank you!

I'll be on vacation for a week and merge your pull request, when I'm back.

Regards, Jörg

Am 01.09.2013 11:59, schrieb Hans Bausewein:

I've added:
TestDataReaderFactory.testReadUShortAndReset()
TestDataReaderFactory.testGetDataReaderJDK6GZipped()

... and made DataReaderFactory.readUShortAndReset() protected to make
it accessible for the test class.

Tests run: 177, Failures: 0, Errors: 0, Skipped: 0

—
Reply to this email directly or view it on GitHub
#74 (comment).

[chewiebug]

Hi Hans,

I have integrated your pull request (#75) and now close this issue. If it shouldn't work for you, please reopen it.

Regards, Jörg