Bug #100559 for Inline-Java: [PATCH] Fix some language issues in the manual page.

Wed Nov 26 05:16:54 2014 roam [...] ringlet.net - Ticket created

From:	roam [...] ringlet.net
Subject:	[PATCH] Fix some language issues in the manual page.

Hi, Thanks for writing and maintaining Inline-Java! What do you think about the attached patch that fixes some typos and rewords some phrases in the documentation? We just added it to the Debian package, but it's ultimately your call :) Description: Fix some language issues in the manual page. Author: Peter Pentchev <roam@ringlet.net> Last-Update: 2014-11-25 The patch is tracked in our Git repository at http://anonscm.debian.org/cgit/pkg-perl/packages/libinline-java-perl.git/plain/debian/patches/01-typos.patch Thanks for considering, Peter Pentchev, Debian Perl Group

Wed Nov 26 05:16:56 2014 roam [...] ringlet.net - Correspondence added

Here's the patch.

Message body is not shown because sender requested not to inline it.

Wed Nov 26 05:16:56 2014 The RT System itself - Status changed from 'new' to 'open'

Wed Nov 26 05:28:31 2014 ETJ [...] cpan.org - Correspondence added

Thanks for this. Most of it looks great. Can you help me with why the 8th chunk (starting "=for comment") is there? Looks like it deletes some important stuff. Is that stuff wrong?

Wed Nov 26 05:41:35 2014 roam [...] ringlet.net - Correspondence added

На ср ное 26 05:28:31 2014, ETJ написа: Show quoted text

> Thanks for this.

Thanks for the lightning-fast reply! Show quoted text

> Most of it looks great. Can you help me with why the > 8th chunk (starting "=for comment") is there? Looks like it deletes > some important stuff. Is that stuff wrong?

Ah, yeah, I intended to add a couple of words about this, but then I forgot. The text that this chunk replaces is copied over from the previous example. I believe that the original intent was to copy the example and the comment, fix the example to demonstrate something else, then fix the comment to document the new example. The last part seems to have slipped through the cracks :) If you take a look at the full Java.pod, you'll see that exactly the same comment is present about 40 lines above; the new text of the chunk is what I believe should document this particular example - but of course I could be wrong and you should take a look at the text to see exactly how wrong it is :) G'luck, Peter

Sat Nov 29 02:14:16 2014 ETJ [...] cpan.org - Correspondence added 10 min

RT-Send-CC:

roam [...] ringlet.net

Thanks for the clarification. Incorporated in new release 0.57.

Sat Nov 29 02:14:20 2014 ETJ [...] cpan.org - Status changed from 'open' to 'resolved'

Sat Nov 29 02:14:20 2014 ETJ [...] cpan.org - Taken