[Zorba-coders] [Merge] lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/trace_without_debug_info-2012-07-13T05-56-14.18Z/log.html -- https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug900677 into lp:zorba
Also, the documentation should say that the function is not yet implemented for general indexes, and the implementation should throw an error in this case. -- https://code.launchpad.net/~zorba-coders/zorba/bug900677/+merge/88973 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/add-xmlid-test into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/add-xmlid-test/+merge/109462 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/add-xmlid-test into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/add-xmlid-test into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/add-xmlid-test/+merge/109462 -- https://code.launchpad.net/~zorba-coders/zorba/add-xmlid-test/+merge/109462 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba
The attempt to merge lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 (message): Validation queue job trace_without_debug_info-2012-07-13T05-56-14.18Z is finished. The final status was: 1 tests did not succeed - changes not commited. Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake -- https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba
The proposal to merge lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 -- https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/add-xmlid-test into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/add-xmlid-test-2012-07-13T06-23-07.103Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/add-xmlid-test/+merge/109462 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/add-xmlid-test into lp:zorba
Validation queue job add-xmlid-test-2012-07-13T06-23-07.103Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/add-xmlid-test/+merge/109462 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/add-xmlid-test into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/add-xmlid-test into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/add-xmlid-test/+merge/109462 -- https://code.launchpad.net/~zorba-coders/zorba/add-xmlid-test/+merge/109462 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 947079] Re: FOTS: GroupBy test results in Seg Fault
Checked in r10934 and the problem was fixed: marking the bug as INVALID.
** Changed in: zorba
Status: New = Invalid
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/947079
Title:
FOTS: GroupBy test results in Seg Fault
Status in Zorba - The XQuery Processor:
Invalid
Bug description:
The following test from the
prod/GroupByClause.xml
results in a Segmentation Fault :
- group-005
xquery version '3.0';
declare namespace o = 'http://www.zorba-xquery.com/options/features';
declare option o:enable 'hof';
for $x in //employee
let $key := $x/hours
group by $key
return group hours={$key} avHours={avg($x/hours)}/
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/947079/+subscriptions
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1024279] [NEW] fn:path seg faults
Public bug reported:
The following tests for fn:path result in a Seg Fault:
Test 1:
fn:path(text{fred})
Test 2:
fn:path(a b=c//@b)
Test 3:
fn:path(ab/b//a/(b[2]))
I have assigned the bug to Markos because he made the most updates to the
fn:path runtime iterator in
src/functions/pregenerated/func_nodes.h.
Bug marked as CRITICAL because Zorba Seg Faults.
** Affects: zorba
Importance: Critical
Assignee: Markos Zaharioudakis (markos-za)
Status: New
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/1024279
Title:
fn:path seg faults
Status in Zorba - The XQuery Processor:
New
Bug description:
The following tests for fn:path result in a Seg Fault:
Test 1:
fn:path(text{fred})
Test 2:
fn:path(a b=c//@b)
Test 3:
fn:path(ab/b//a/(b[2]))
I have assigned the bug to Markos because he made the most updates to the
fn:path runtime iterator in
src/functions/pregenerated/func_nodes.h.
Bug marked as CRITICAL because Zorba Seg Faults.
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/1024279/+subscriptions
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/no_disqus into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/no_disqus into lp:zorba has been updated. Commit Message changed to: Remove disqus references For more details, see: https://code.launchpad.net/~zorba-coders/zorba/no_disqus/+merge/114793 -- https://code.launchpad.net/~zorba-coders/zorba/no_disqus/+merge/114793 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba
The proposal to merge lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 -- https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/trace_without_debug_info-2012-07-13T08-33-57.278Z/log.html -- https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/no_disqus into lp:zorba
What process will add the Disqus stuff back in when creating the website? -- https://code.launchpad.net/~zorba-coders/zorba/no_disqus/+merge/114793 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1024279] Re: fn:path seg faults
*** This bug is a duplicate of bug 1024123 ***
https://bugs.launchpad.net/bugs/1024123
A branch that will hopefully fix these problems exists already and waits
review from Ghislain. There is already another bug for this (1024123).
** This bug has been marked a duplicate of bug 1024123
Un-revert r10899
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/1024279
Title:
fn:path seg faults
Status in Zorba - The XQuery Processor:
New
Bug description:
The following tests for fn:path result in a Seg Fault:
Test 1:
fn:path(text{fred})
Test 2:
fn:path(a b=c//@b)
Test 3:
fn:path(ab/b//a/(b[2]))
I have assigned the bug to Markos because he made the most updates to the
fn:path runtime iterator in
src/functions/pregenerated/func_nodes.h.
Bug marked as CRITICAL because Zorba Seg Faults.
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/1024279/+subscriptions
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1023120-schemaImport into lp:zorba
Review: Approve Change looks good to me. I also tested it with a bigger schema and with valgrind. I don't see a memory problem. -- https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba
Validation queue job trace_without_debug_info-2012-07-13T08-33-57.278Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba
The proposal to merge lp:~davidagraf/zorba/trace_without_debug_info into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 -- https://code.launchpad.net/~davidagraf/zorba/trace_without_debug_info/+merge/110377 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114739 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
Review: Needs Fixing Hi Till, I noticed after approving that the diff seems to replace the new EQName syntax with the old one. Is this the intent? -- https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114739 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1023862 into lp:zorba
Why didn't you just assign the bug to me? (Or why don't you just assign it to me now?) did that. Thanks for helping. -- https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114621 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1023862] Re: file:list($path, true()) can produce infinite loops on some filesystems
** Changed in: zorba Assignee: Dennis Knochenwefel (dennis-knochenwefel) = Paul J. Lucas (paul-lucas) -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/1023862 Title: file:list($path, true()) can produce infinite loops on some filesystems Status in Zorba - The XQuery Processor: In Progress Bug description: in fs_util the next() function makes use of ::readdir and in particular d_type which is not supported by all file system types (cf readdir man page). If d_type is not supported it will be set to DT_UNKNOWN also for the file system entries . and ... Consequently, these entries are not skipped and file:list will recursively infinitely list entries: /. /./. /././. /./././. /././././. /./././././. .. etc To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/1023862/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~davidagraf/zorba/bug-867248 into lp:zorba/email-module
The proposal to merge lp:~davidagraf/zorba/bug-867248 into lp:zorba/email-module has been updated. Commit Message changed to: Fix for bug #867248. For more details, see: https://code.launchpad.net/~davidagraf/zorba/bug-867248/+merge/113059 -- https://code.launchpad.net/~davidagraf/zorba/bug-867248/+merge/113059 Your team Zorba Coders is subscribed to branch lp:zorba/email-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~davidagraf/zorba/bug-867248 into lp:zorba/email-module
The proposal to merge lp:~davidagraf/zorba/bug-867248 into lp:zorba/email-module has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~davidagraf/zorba/bug-867248/+merge/113059 -- https://code.launchpad.net/~davidagraf/zorba/bug-867248/+merge/113059 Your team Zorba Coders is subscribed to branch lp:zorba/email-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~davidagraf/zorba/bug-867248 into lp:zorba/email-module
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-867248-2012-07-13T11-05-46.439Z/log.html -- https://code.launchpad.net/~davidagraf/zorba/bug-867248/+merge/113059 Your team Zorba Coders is subscribed to branch lp:zorba/email-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~davidagraf/zorba/bug-867248 into lp:zorba/email-module
Validation queue job bug-867248-2012-07-13T11-05-46.439Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~davidagraf/zorba/bug-867248/+merge/113059 Your team Zorba Coders is subscribed to branch lp:zorba/email-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1022762] Re: Multi-char escapes wrongly forbidden in character class
** Changed in: zorba Status: In Progress = Fix Committed -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/1022762 Title: Multi-char escapes wrongly forbidden in character class Status in Zorba - The XQuery Processor: Fix Committed Bug description: If you have a character range, e.g., A-Z, then the end-point chars in the range can be SingleCharEsc. A while ago, a fix was made for this, but the fix went too far and forbids MultiCharEsc within charClassExpr. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/1022762/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1023862 into lp:zorba
Paul J. Lucas has proposed merging lp:~zorba-coders/zorba/bug-1023862 into
lp:zorba.
Requested reviews:
Dennis Knochenwefel (dennis-knochenwefel)
Paul J. Lucas (paul-lucas)
Related bugs:
Bug #1023862 in Zorba: file:list($path, true()) can produce infinite loops
on some filesystems
https://bugs.launchpad.net/zorba/+bug/1023862
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881
Added case for DT_UNKNOWN.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/util/fs_util.cpp'
--- src/util/fs_util.cpp 2012-07-12 17:29:55 +
+++ src/util/fs_util.cpp 2012-07-13 14:46:21 +
@@ -53,6 +53,16 @@
// helper functions ///
+#ifndef WIN32
+inline bool is_dots( char const *s ) {
+ return s[0] == '.' (!s[1] || (s[1] == '.' !s[2]));
+}
+#else
+inline bool is_dots( LPCWSTR s ) {
+ return s[0] == TEXT('.') (!s[1] || (s[1] == TEXT('.') !s[2]));
+}
+#endif /* WIN32 */
+
inline void replace_foreign( zstring *path ) {
#ifdef WIN32
ascii::replace_all( *path, '/', '\\' );
@@ -323,9 +333,8 @@
switch ( ent_-d_type ) {
case DT_DIR: {
char const *const name = ent_-d_name;
- // skip . and .. entries
- if ( name[0] == '.' (!name[1] || (name[1] == '.' !name[2])) )
-continue;
+ if ( is_dots( name ) )
+continue; // skip . and .. entries
ent_type_ = directory;
break;
}
@@ -335,6 +344,21 @@
case DT_REG:
ent_type_ = file;
break;
+case DT_UNKNOWN: {
+ //
+ // The d_type member is not supported by all filesystems. If it's
+ // not, it will be set to DT_UNKNOWN. Hence, we're forced to do an
+ // explicit stat() to determine the entry type.
+ //
+ // This check fixes bug #1023862.
+ //
+ zstring ent_path( dir_path_ );
+ fs::append( ent_path, static_castchar const*( ent_-d_name ) );
+ ent_type_ = get_type( ent_path );
+ if ( ent_type_ == directory is_dots( ent_-d_name ) )
+continue; // skip . and .. entries
+ break;
+}
default:
ent_type_ = other;
}
@@ -352,11 +376,8 @@
}
LPCWSTR const wname = ent_data_.cFileName;
- // skip . and .. entries
- if ( wname[0] == TEXT('.')
- (!wname[1] || wname[1] == TEXT('.') !wname[2]) )
-continue;
-
+ if ( is_dots( wname ) )
+continue; // skip . and .. entries
win32::to_char( wname, ent_name_ );
ent_type_ = win32::map_type( ent_data_.dwFileAttributes );
return true;
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1023862 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1024279] Re: fn:path seg faults
*** This bug is a duplicate of bug 1024123 ***
https://bugs.launchpad.net/bugs/1024123
** Branch linked: lp:~zorba-coders/zorba/fix-fn-path
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/1024279
Title:
fn:path seg faults
Status in Zorba - The XQuery Processor:
New
Bug description:
The following tests for fn:path result in a Seg Fault:
Test 1:
fn:path(text{fred})
Test 2:
fn:path(a b=c//@b)
Test 3:
fn:path(ab/b//a/(b[2]))
I have assigned the bug to Markos because he made the most updates to the
fn:path runtime iterator in
src/functions/pregenerated/func_nodes.h.
Bug marked as CRITICAL because Zorba Seg Faults.
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/1024279/+subscriptions
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1023120-schemaImport into lp:zorba
Fixed. Please check. -- https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1024448] [NEW] data-converter module problems with non utf-8 characters
Public bug reported:
In public Json streams lots of non-utf8 character escapes can be found
causing some problems when parsing json or tidying the contained html (
as for example marketed here: http://www.charbase.com/1f44a-unicode-
fisted-hand-sign ).
The following example Query causes a whole bunch of problems:
import module namespace json =
http://www.zorba-xquery.com/modules/converters/json;;
import module namespace html =
http://www.zorba-xquery.com/modules/converters/html;;
declare namespace j = http://john.snelson.org.uk/parsing-json-into-xquery;;
let $text := lt;p || json:parse({text:Let's get it.
\ud83d\udc4a})/j:pair[@name=text]/text() || lt;/p
return html:parse($text)
Problems:
1. html:parse () has return type document-node(), but tries to return an
empty-sequence in this example (discovered by ghislain)
2. in file
src/com/zorba-xquery/www/modules/converters/html.xq.src/tidy_wrapper.h function
createHtmlItem(...) doesn't throw a proper error message (discovered by
ghislain) which makes debugging really hard. In contrast, parse-xml throws a
very helpful error:
dynamic error [err:FODC0006]: invalid content passed to fn:parse-xml():
loader parsing error: Char 0xD83D out of allowed range;
Could html:parse report the same error?
3. json:parse() doesn't report an error here which is good in my
opinion. Yet, as these utf-16 (?) encoded characters are used a lot in
json, would it be possible to transform them into valid utf-8 (e.g.
\ud83d\udc4a - #x1f44a;)?
Maybe these findings are going to be a problem in Jsoniq as well?
** Affects: zorba
Importance: High
Assignee: Paul J. Lucas (paul-lucas)
Status: New
** Tags: improve-code-quality incorrect-result jsoniq usability
** Changed in: zorba
Importance: Undecided = High
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/1024448
Title:
data-converter module problems with non utf-8 characters
Status in Zorba - The XQuery Processor:
New
Bug description:
In public Json streams lots of non-utf8 character escapes can be found
causing some problems when parsing json or tidying the contained html
( as for example marketed here: http://www.charbase.com/1f44a-unicode-
fisted-hand-sign ).
The following example Query causes a whole bunch of problems:
import module namespace json =
http://www.zorba-xquery.com/modules/converters/json;;
import module namespace html =
http://www.zorba-xquery.com/modules/converters/html;;
declare namespace j = http://john.snelson.org.uk/parsing-json-into-xquery;;
let $text := lt;p || json:parse({text:Let's get it.
\ud83d\udc4a})/j:pair[@name=text]/text() || lt;/p
return html:parse($text)
Problems:
1. html:parse () has return type document-node(), but tries to return
an empty-sequence in this example (discovered by ghislain)
2. in file
src/com/zorba-xquery/www/modules/converters/html.xq.src/tidy_wrapper.h function
createHtmlItem(...) doesn't throw a proper error message (discovered by
ghislain) which makes debugging really hard. In contrast, parse-xml throws a
very helpful error:
dynamic error [err:FODC0006]: invalid content passed to fn:parse-xml():
loader parsing error: Char 0xD83D out of allowed range;
Could html:parse report the same error?
3. json:parse() doesn't report an error here which is good in my
opinion. Yet, as these utf-16 (?) encoded characters are used a lot in
json, would it be possible to transform them into valid utf-8 (e.g.
\ud83d\udc4a - #x1f44a;)?
Maybe these findings are going to be a problem in Jsoniq as well?
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/1024448/+subscriptions
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1024448] Re: data-converter module problems with non utf-8 characters
If there are 3 problems, there should be 3 different bugs, not all
lumped together into a single bug. I have nothing to do with either
html:parse() or tidy.
As for the 3rd bug, you don't say what the error is that it should
report. What exactly is wrong with JSON parsing?
** Changed in: zorba
Status: New = Incomplete
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/1024448
Title:
data-converter module problems with non utf-8 characters
Status in Zorba - The XQuery Processor:
Incomplete
Bug description:
In public Json streams lots of non-utf8 character escapes can be found
causing some problems when parsing json or tidying the contained html
( as for example marketed here: http://www.charbase.com/1f44a-unicode-
fisted-hand-sign ).
The following example Query causes a whole bunch of problems:
import module namespace json =
http://www.zorba-xquery.com/modules/converters/json;;
import module namespace html =
http://www.zorba-xquery.com/modules/converters/html;;
declare namespace j = http://john.snelson.org.uk/parsing-json-into-xquery;;
let $text := lt;p || json:parse({text:Let's get it.
\ud83d\udc4a})/j:pair[@name=text]/text() || lt;/p
return html:parse($text)
Problems:
1. html:parse () has return type document-node(), but tries to return
an empty-sequence in this example (discovered by ghislain)
2. in file
src/com/zorba-xquery/www/modules/converters/html.xq.src/tidy_wrapper.h function
createHtmlItem(...) doesn't throw a proper error message (discovered by
ghislain) which makes debugging really hard. In contrast, parse-xml throws a
very helpful error:
dynamic error [err:FODC0006]: invalid content passed to fn:parse-xml():
loader parsing error: Char 0xD83D out of allowed range;
Could html:parse report the same error?
3. json:parse() doesn't report an error here which is good in my
opinion. Yet, as these utf-16 (?) encoded characters are used a lot in
json, would it be possible to transform them into valid utf-8 (e.g.
\ud83d\udc4a - #x1f44a;)?
Maybe these findings are going to be a problem in Jsoniq as well?
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/1024448/+subscriptions
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1023862 into lp:zorba
Review: Approve there was a change in src/store/naive/item.cpp that made the build fail on windows. After removing that it built fine and all file tests known to pass passed. perfect. -- https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1023862 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-1023862 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1023862 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1023862-2012-07-13T17-24-06.246Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1023862 into lp:zorba
Validation queue job bug-1023862-2012-07-13T17-24-06.246Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1023862 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-1023862 into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1023862/+merge/114881 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
Hi Till, I noticed after approving that the diff seems to replace the new EQName syntax with the old one. Is this the intent? No, it isn't. Thanks! -- https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114739 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
Till Westmann has proposed merging lp:~zorba-coders/zorba/fix-fn-path into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Markos Zaharioudakis (markos-za)
Related bugs:
Bug #1024123 in Zorba: Un-revert r10899
https://bugs.launchpad.net/zorba/+bug/1024123
Bug #1024279 in Zorba: fn:path seg faults
https://bugs.launchpad.net/zorba/+bug/1024279
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922
fix position for elements and processing-instructions in fn:path
--
https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/runtime/nodes/nodes_impl.cpp'
--- src/runtime/nodes/nodes_impl.cpp 2012-07-12 17:29:55 +
+++ src/runtime/nodes/nodes_impl.cpp 2012-07-13 18:59:22 +
@@ -680,7 +680,7 @@
/***
/
-int getNodePosition(store::Item_t aNode)
+int getNodePosition(store::Item_t aNode, store::Item_t aNodeName)
{
int count = 1;
store::Iterator_t lIterator = aNode-getParent()-getChildren();
@@ -692,7 +692,7 @@
{
if(lItem-equals(aNode))
break;
- else
+ else if (aNodeName.isNull() || aNodeName-equals(lItem-getNodeName()))
count++;
}
}
@@ -703,101 +703,87 @@
bool FnPathIterator::nextImpl(store::Item_t result, PlanState planState) const
{
store::Item_t inNode;
+ store::Item_t swap;
store::Item_t nodeName;
- store::NsBindings nsBindings;
zstring path;
zstring temp;
zstring zNamespace;
zstring zLocalName;
zstring zPosition;
- bool rootIsDocument = false;
PlanIteratorState* state;
DEFAULT_STACK_INIT(PlanIteratorState, state, planState);
if (consumeNext(inNode, theChildren[0], planState))
{
-do
-{
+while (inNode-getParent())
+{
+ temp = path;
+ path = /;
+
switch (inNode-getNodeKind())
{
-case store::StoreConsts::documentNode:
- temp = path;
- path = /;
- path += temp;
- rootIsDocument = true;
- break;
-
case store::StoreConsts::elementNode:
nodeName = inNode-getNodeName();
zNamespace = nodeName-getNamespace();
zLocalName = nodeName-getLocalName();
- zPosition = ztd::to_string(getNodePosition(inNode));
- temp = path;
- path = Q{+zNamespace+}+zLocalName+[+zPosition.c_str()+];
- path += temp;
+ zPosition = ztd::to_string(getNodePosition(inNode, nodeName));
+ path += Q{ + zNamespace + } + zLocalName + [ + zPosition + ];
break;
-
case store::StoreConsts::attributeNode:
nodeName = inNode-getNodeName();
zNamespace =nodeName-getNamespace();
zLocalName = nodeName-getLocalName();
+ path += @;
if(zNamespace != )
{
-temp = path;
-path = @Q{+zNamespace+}+zLocalName;
-path += temp;
- }
- else
- {
-temp = path;
-path = @+zLocalName;
-path += temp;
- }
+path += Q{ + zNamespace + };
+ }
+ path += zLocalName;
break;
-
case store::StoreConsts::textNode:
- zPosition = ztd::to_string(getNodePosition(inNode));
- temp = path;
- path = text()[+zPosition+];
- path += temp;
+ zPosition = ztd::to_string(getNodePosition(inNode, NULL));
+ path += text()[ + zPosition + ];
break;
-
case store::StoreConsts::commentNode:
- zPosition = ztd::to_string(getNodePosition(inNode));
- temp = path;
- path = comment()[+zPosition+];
- path += temp;
+ zPosition = ztd::to_string(getNodePosition(inNode, NULL));
+ path += comment()[ + zPosition + ];
break;
-
-case store::StoreConsts::piNode:
+case store::StoreConsts::piNode:
nodeName = inNode-getNodeName();
zLocalName = nodeName-getLocalName();
- zPosition = ztd::to_string(getNodePosition(inNode));
- temp = path;
- path = processing-instruction(+zLocalName+)[+zPosition+];
- path += temp;
+ zPosition = ztd::to_string(getNodePosition(inNode, nodeName));
+ path += processing-instruction( + zLocalName + )[ + zPosition + ];
break;
-
default:
+ // this is either a documentNode which should always be a root
+ // node (and not end up here) or it is something very strange
ZORBA_ASSERT(false);
+ break;
}
-
- inNode = inNode-getParent();
-
- if (inNode inNode-getNodeKind() != store::StoreConsts::documentNode)
+ path += temp;
+
+ swap = inNode-getParent();
+
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
Resubmitted proposal and added Matthias as reviewer. -- https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-fn-path into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922 -- https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-fn-path into lp:zorba has been updated. Commit Message changed to: fix position for elements and processing-instructions in fn:path For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922 -- https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
Validation queue job fix-fn-path-2012-07-13T19-06-57.88Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix-fn-path into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922 -- https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba
The proposal to merge lp:~nbrinza/zorba/error-messages into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114935 -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114935 Your team Zorba Coders is requested to review the proposed merge of lp:~nbrinza/zorba/error-messages into lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/error-messages-2012-07-13T20-03-58.646Z/log.html -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114935 Your team Zorba Coders is requested to review the proposed merge of lp:~nbrinza/zorba/error-messages into lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba
Validation queue job error-messages-2012-07-13T20-03-58.646Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114935 Your team Zorba Coders is requested to review the proposed merge of lp:~nbrinza/zorba/error-messages into lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 1 Pending. -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/114935 Your team Zorba Coders is requested to review the proposed merge of lp:~nbrinza/zorba/error-messages into lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-fn-path into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix-fn-path/+merge/114922 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1024580] [NEW] Error with precompiled headers in Windows
Public bug reported: Windows is not compiling on VS IDE with precompiled headers activated... ** Affects: zorba Importance: Undecided Status: New -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/1024580 Title: Error with precompiled headers in Windows Status in Zorba - The XQuery Processor: New Bug description: Windows is not compiling on VS IDE with precompiled headers activated... To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/1024580/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1024580 into lp:zorba
Rodolfo Ochoa has proposed merging lp:~zorba-coders/zorba/bug1024580 into lp:zorba. Requested reviews: Cezar Andrei (cezar-andrei) Chris Hillery (ceejatec) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 Fix for precompiled headers on Windows -- https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'src/compiler/expression/expr_utils.cpp' --- src/compiler/expression/expr_utils.cpp 2012-05-23 21:43:47 + +++ src/compiler/expression/expr_utils.cpp 2012-07-14 00:42:20 + @@ -13,7 +13,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ - +#include stdafx.h #include compiler/expression/expr_utils.h #include zorbaserialization/serialize_basic_types.h === modified file 'src/compiler/expression/json_exprs.cpp' --- src/compiler/expression/json_exprs.cpp 2012-07-11 15:38:39 + +++ src/compiler/expression/json_exprs.cpp 2012-07-14 00:42:20 + @@ -13,7 +13,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ - +#include stdafx.h #include compiler/expression/json_exprs.h #include compiler/expression/expr_visitor.h === modified file 'src/precompiled/stdafx.h' --- src/precompiled/stdafx.h 2012-07-12 17:29:55 + +++ src/precompiled/stdafx.h 2012-07-14 00:42:20 + @@ -72,10 +72,8 @@ #include zorbautils/hashfun.h #include zorbautils/hashmap.h #include zorbautils/hashmap_itemp.h - #include zorbautils/hashmap_str_obj.h #include zorbautils/hashmap_zstring.h #include zorbautils/hashset.h - #include zorbautils/hashset_itemh.h #include zorbautils/latch.h #include zorbautils/locale.h #include zorbautils/lock.h === modified file 'src/runtime/full_text/ft_module_impl.cpp' --- src/runtime/full_text/ft_module_impl.cpp 2012-07-12 17:29:55 + +++ src/runtime/full_text/ft_module_impl.cpp 2012-07-14 00:42:20 + @@ -14,6 +14,7 @@ * limitations under the License. */ +#include stdafx.h #include zorba/config.h // @@ -23,7 +24,6 @@ // #ifndef ZORBA_NO_FULL_TEXT -#include stdafx.h #include limits #include typeinfo === modified file 'src/runtime/full_text/ft_module_util.cpp' --- src/runtime/full_text/ft_module_util.cpp 2012-07-11 15:38:39 + +++ src/runtime/full_text/ft_module_util.cpp 2012-07-14 00:42:20 + @@ -14,6 +14,7 @@ * limitations under the License. */ +#include stdafx.h #include api/unmarshaller.h #include context/static_context.h #include store/api/item_factory.h === modified file 'src/runtime/json/jsoniq_functions_impl.cpp' --- src/runtime/json/jsoniq_functions_impl.cpp 2012-07-11 15:38:39 + +++ src/runtime/json/jsoniq_functions_impl.cpp 2012-07-14 00:42:20 + @@ -13,11 +13,11 @@ * See the License for the specific language governing permissions and * limitations under the License. */ +#include stdafx.h #include zorba/config.h #ifdef ZORBA_WITH_JSON -#include stdafx.h #include sstream === modified file 'src/util/base64_util.cpp' --- src/util/base64_util.cpp 2012-06-20 22:26:09 + +++ src/util/base64_util.cpp 2012-07-14 00:42:20 + @@ -14,6 +14,7 @@ * limitations under the License. */ +#include stdafx.h #include algorithm #include cstring -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1024580 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug1024580 into lp:zorba has been updated. Commit Message changed to: Fix for precompiled headers on Windows For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 -- https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1024580 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1024580] Re: Error with precompiled headers in Windows
** Branch linked: lp:~zorba-coders/zorba/bug1024580 -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/1024580 Title: Error with precompiled headers in Windows Status in Zorba - The XQuery Processor: New Bug description: Windows is not compiling on VS IDE with precompiled headers activated... To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/1024580/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1024580 into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1024580 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug1024580 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 -- https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1024580 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug1024580-2012-07-14T01-20-15.753Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1023120-schemaImport into lp:zorba
Review: Approve I wish Xerces' doc confirmed that setPublicId() and setSystemId() made local copies of those parameters, but valgrind reports no leaks so clearly they do! -- https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1023120-schemaImport into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug1023120-schemaImport into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712 -- https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1024580 into lp:zorba
Validation queue job bug1024580-2012-07-14T01-20-15.753Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1024580 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug1024580 into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 -- https://code.launchpad.net/~zorba-coders/zorba/bug1024580/+merge/114966 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1023120-schemaImport into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug1023120-schemaImport-2012-07-14T01-48-00.078Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 1024580] Re: Error with precompiled headers in Windows
** Changed in: zorba Status: New = Fix Committed -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/1024580 Title: Error with precompiled headers in Windows Status in Zorba - The XQuery Processor: Fix Committed Bug description: Windows is not compiling on VS IDE with precompiled headers activated... To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/1024580/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1023120-schemaImport into lp:zorba
Validation queue job bug1023120-schemaImport-2012-07-14T01-48-00.078Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug1023120-schemaImport into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug1023120-schemaImport into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712 -- https://code.launchpad.net/~zorba-coders/zorba/bug1023120-schemaImport/+merge/114712 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
