Merge lp:~zorba-coders/zorba/windows-fixes into lp:zorba

Proposed by Matthias Brantner
Status: Merged
Approved by: Chris Hillery
Approved revision: no longer in the source branch.
Merged at revision: 10849
Proposed branch: lp:~zorba-coders/zorba/windows-fixes
Merge into: lp:zorba
Diff against target: 28 lines (+5/-2)
2 files modified
src/store/naive/simple_item_factory.cpp (+1/-1)
test/unit/CMakeLists.txt (+4/-1)
To merge this branch: bzr merge lp:~zorba-coders/zorba/windows-fixes
Reviewer Review Type Date Requested Status
Chris Hillery Approve
Matthias Brantner Approve
Review via email: mp+106083@code.launchpad.net

Commit message

- fixed itemfactory unit test on Windows
- disabled string unit test on Windows (because of bug #867271)

Description of the change

- fixed itemfactory unit test on Windows
- disabled string unit test on Windows (because of bug #867271)

To post a comment you must log in.
Revision history for this message
Matthias Brantner (matthias-brantner) :
review: Approve
Revision history for this message
Chris Hillery (ceejatec) :
review: Approve
lp:~zorba-coders/zorba/windows-fixes updated
10848. By Chris Hillery

Fix (from Matthias) to ensure StaticContext used for invoke() lives as long as the ItemSequence returned from said invoke().
 Approved: Matthias Brantner, Chris Hillery

Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue job windows-fixes-2012-05-17T01-11-08.33Z is finished. The final status was:

All tests succeeded!

lp:~zorba-coders/zorba/windows-fixes updated
10849. By Matthias Brantner

- fixed itemfactory unit test on Windows
- disabled string unit test on Windows (because of bug #867271) Approved: Chris Hillery, Matthias Brantner

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/store/naive/simple_item_factory.cpp'
2--- src/store/naive/simple_item_factory.cpp 2012-05-08 01:09:52 +0000
3+++ src/store/naive/simple_item_factory.cpp 2012-05-17 00:53:18 +0000
4@@ -1011,7 +1011,7 @@
5 xs_base64Binary value)
6 {
7 const std::vector<char>& data = value.getData();
8- result = new Base64BinaryItem(&data[0], data.size(), true);
9+ result = new Base64BinaryItem(data.size()!=0?&data[0]:0, data.size(), true);
10 return true;
11 }
12
13
14=== modified file 'test/unit/CMakeLists.txt'
15--- test/unit/CMakeLists.txt 2012-05-03 12:31:51 +0000
16+++ test/unit/CMakeLists.txt 2012-05-17 00:53:18 +0000
17@@ -137,7 +137,10 @@
18 ZORBA_GENERATE_EXE(LibUnitTest lib_unit_test.cpp "" "" "")
19
20 # ADD NEW UNIT TESTS HERE
21-ZORBA_ADD_TEST("test/libunit/string" LibUnitTest string)
22+IF (NOT WIN32)
23+ # disabled because of bug lp:867271
24+ ZORBA_ADD_TEST("test/libunit/string" LibUnitTest string)
25+ENDIF (NOT WIN32)
26 ZORBA_ADD_TEST("test/libunit/uri" LibUnitTest uri)
27 EXPECTED_FAILURE(test/libunit/uri 3118348)
28 ZORBA_ADD_TEST("test/libunit/json_parser" LibUnitTest json_parser)

Subscribers

People subscribed via source and target branches