Merge lp:~zorba-coders/zorba/bug-921458 into lp:zorba
Proposed by
Matthias Brantner
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Paul J. Lucas | ||||
Approved revision: | 10755 | ||||
Merged at revision: | 10766 | ||||
Proposed branch: | lp:~zorba-coders/zorba/bug-921458 | ||||
Merge into: | lp:zorba | ||||
Diff against target: |
291 lines (+222/-5) 7 files modified
ChangeLog (+1/-0) modules/org/expath/ns/file.xq (+1/-5) modules/org/expath/ns/file.xq.src/file.cpp (+118/-0) modules/org/expath/ns/file.xq.src/file.h (+64/-0) modules/org/expath/ns/file.xq.src/file_module.cpp (+2/-0) test/rbkt/ExpQueryResults/zorba/file/file_read_text_lines.xml.res (+32/-0) test/rbkt/Queries/zorba/file/file_read_text_lines.xq (+4/-0) |
||||
To merge this branch: | bzr merge lp:~zorba-coders/zorba/bug-921458 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paul J. Lucas | Approve | ||
William Candillon | Approve | ||
Review via email: mp+102058@code.launchpad.net |
This proposal supersedes a proposal from 2012-04-13.
Commit message
Fix for bug #921458 (file:read-
Description of the change
Fix for bug #921458 (file:read-
The problem was that the string:split function isn't suitable for tokenizing with newlines as separator.
I have provided a native implementation using C++'s getline.
To post a comment you must log in.
Validation queue starting for merge proposal. zorbatest. lambda. nu:8080/ remotequeue/ bug-921458- 2012-04- 13T09-53- 08.237Z/ log.html
Log at: http://