[ 
https://issues.apache.org/jira/browse/ANY23-93?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13280769#comment-13280769
 ] 

Hudson commented on ANY23-93:
-----------------------------

Integrated in Any23-trunk #209 (See 
[https://builds.apache.org/job/Any23-trunk/209/])
    Fixed issue #ANY23-93. (Revision 1341040)

     Result = SUCCESS
mostarda : 
Files : 
* 
/incubator/any23/trunk/core/src/main/java/org/apache/any23/io/nquads/NQuadsParser.java
* 
/incubator/any23/trunk/core/src/test/java/org/apache/any23/io/nquads/NQuadsParserTest.java

                
> NQuadsParser column numbers after first row are 0-index-based instead of 
> 1-index-based
> --------------------------------------------------------------------------------------
>
>                 Key: ANY23-93
>                 URL: https://issues.apache.org/jira/browse/ANY23-93
>             Project: Apache Any23
>          Issue Type: Bug
>            Reporter: Peter Ansell
>
> nextRow has a bug currently where it resets the column number to 0 instead of 
> 1, which results in incorrect column numbers after the first row. The first 
> row is okay as it is setup initially to 1 in the parse method before nextRow 
> is called.
> Patch is a simple edit to modify nextRow to start like:
>     private void nextRow() {
>         col = 1;
> There will also be edits needed to the test suite to fix expected column 
> numbers that are then off by one in testParserWithAllCases and 
> testParserWithRealData

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to