Merge lp:~matthias-brantner/zorba/bug-fixing into lp:zorba

[Matthias Brantner]

- fixed DynamicLoader memory leak
- StaticContext::getAuditEvent const
- added iterator.h include to zorba.h

- fixed DynamicLoader memory leak
- StaticContext::getAuditEvent const
- added iterator.h include to zorba.h

[Matthias Brantner (matthias-brantner)]

review: Approve

[lp:~matthias-brantner/zorba/bug-fixing]

10549. By Matthias Brantner on 2011-11-22

Following redirects when resolving URLs. Mostly used when importing modules or schemas. Approved: Chris Hillery, Matthias Brantner

10550. By Daniel Turcanu on 2011-11-22

Fixed bug #872850 about plan serializer crash Approved: David Graf, Matthias Brantner

10551. By Carlos Manuel Lopez on 2011-11-22

Optimization of the fn:substring function in the case when its $start and $length arguments are integers. Approved: Matthias Brantner, Markos Zaharioudakis

10552. By Paul J. Lucas on 2011-11-22

Removed warnings. Approved: Matthias Brantner, Paul J. Lucas

10553. By Markos Zaharioudakis on 2011-11-22

converted variable assignment to a full-blown expression instead of a function + fixed a bug with the cloning of var_decl_expr Approved: Markos Zaharioudakis

10554. By Matthias Brantner on 2011-11-23

- fixed DynamicLoader memory leak
- StaticContext::getAuditEvent const
- added iterator.h include to zorba.h
- fn:trace should not be constant folded and propagate the return type of its first argument Approved: Markos Zaharioudakis, Matthias Brantner

10555. By Matthias Brantner on 2011-11-23

- Reorganization of ChangeLog for 2.1 release
- added the xqxq-module to the modules that are downloaded automatically Approved: Chris Hillery, Matthias Brantner

10556. By Matthias Brantner on 2011-11-25

Allow external functions to be updating. Approved: Markos Zaharioudakis, Juan Zacarias

10557. By Markos Zaharioudakis on 2011-11-29

fixed a performance bug in the implementation of groupby: no node copying for non-groupby vars Approved: Matthias Brantner, Markos Zaharioudakis

10558. By Sorin Marian Nasoi on 2011-11-29

Fix for bug #897619. Approved: Markos Zaharioudakis, Sorin Marian Nasoi

10559. By Sorin Marian Nasoi on 2011-11-29

Fix for bug #897616. Approved: Nicolae Brinza, Sorin Marian Nasoi

10560. By Matthias Brantner on 2011-11-29

Do not raise warnings for unknown annotations in unknown namespaces Approved: Nicolae Brinza, William Candillon, Matthias Brantner

10561. By Juan Zacarias on 2011-11-30

API changes
Removed isBoundExternalVariable from xquery.h
Added 2 functions to dynamic_context.h
isBoundContextItem and isBoundExternalVariable Approved: Matthias Brantner, Markos Zaharioudakis

10562. By Daniel Turcanu on 2011-11-30

fn:analyze-string fix
also, adding config.h include to ensure correct error messages Approved: Chris Hillery, Matthias Brantner

10563. By Matthias Brantner on 2011-11-30

fix for building modules outside of Zorba Approved: Chris Hillery, Matthias Brantner

10564. By William Candillon on 2011-11-30

Fix bug #897825 Approved: Rodolfo Ochoa, Matthias Brantner

10565. By Matthias Brantner on 2011-11-30

Unmarked XQueryX fn-put-005-fail as expected to fail in CMake. Approved: Chris Hillery, Matthias Brantner

10566. By Chris Hillery on 2011-11-30

Update ExternalModules.conf to download modules tagged 'zorba-2.1'. Approved: Matthias Brantner, Chris Hillery

10567. By Chris Hillery on 2011-12-01

fix for installing and using modules on windows Approved: Chris Hillery, Matthias Brantner

10568. By Chris Hillery on 2011-12-01

Separate check_uris into check_core_uris and check_uris. zorbacmd depends on check_core_uris, so non-core modules may depend on zorbacmd and be assured that the Zorba executable is fully functional. Approved: Matthias Brantner, Chris Hillery

10569. By Chris Hillery on 2011-12-01

Fix reversed URI installation directories. Approved: Matthias Brantner, Chris Hillery

10570. By Matthias Brantner on 2011-12-02

fix for bug #898064 Approved: Nicolae Brinza, William Candillon

10571. By Markos Zaharioudakis on 2011-12-05

small optimization of the group by Approved: Markos Zaharioudakis

10572. By Matthias Brantner on 2011-12-05

made destructors of Item subclasses virtual Approved: Markos Zaharioudakis, Matthias Brantner

10573. By Matthias Brantner on 2011-12-05

optimization: only have exactly two instances of boolean items in the simple store Approved: Markos Zaharioudakis, Matthias Brantner

10574. By Chris Hillery on 2011-12-05

Only create check_core_uris and associated dependencies if there are any core URIs (to prevent creating this target in standalone module projects).
 Approved: Juan Zacarias, Chris Hillery

10575. By Paul J. Lucas on 2011-12-06

undo previous changes and mark bug as expected to fail. Approved: Paul J. Lucas, Matthias Brantner

10576. By Paul J. Lucas on 2011-12-06

Things should be reverted now. Approved: Matthias Brantner, Paul J. Lucas

10577. By Matthias Brantner on 2011-12-07

fix build on windows Approved: Juan Zacarias, Matthias Brantner

10578. By Paul J. Lucas on 2011-12-08

Fixed build. Approved: Matthias Brantner, Paul J. Lucas

10579. By Matthias Brantner on 2011-12-09

exclude .bzr dir from source packages Approved: Cezar Andrei, Matthias Brantner

10580. By Markos Zaharioudakis on 2011-12-09

Cleaned up unused stuff in debugger_expr Approved: Gabriel Petrovay, Markos Zaharioudakis

10581. By Matthias Brantner on 2011-12-12

removed obsolete code guarded by the USE_POS macro Approved: Markos Zaharioudakis, Matthias Brantner

10582. By Matthias Brantner on 2011-12-13

- automatic caching of recursive, non-sequential, and deterministic functions with atomic parameter and return types
- %ann:cache and %ann:no-cache for controlling function result caching Approved: Markos Zaharioudakis, Matthias Brantner

10583. By Markos Zaharioudakis on 2011-12-14

Optimized and cleaned up SimpleTempSeq implementation and its usage. Approved: Markos Zaharioudakis

10584. By Paul J. Lucas on 2011-12-15

Applied William's patch; patched William's patch to handle UTF-8 properly. Approved: William Candillon, Paul J. Lucas

Unmerged revisions

=== modified file 'ChangeLog'
--- ChangeLog	2011-11-16 04:00:29 +0000
+++ ChangeLog	2011-11-17 05:05:28 +0000
@@ -29,6 +29,7 @@
     corruption will occur)
   * Fixed bug # (Should not destroy the parent of a node that is being detached
     before the detach is done).
+  * Added const qualifier to StaticContext::getAuditEvent() 
   * Fixed bug #3408181 (available-collection() returns undeclared collections)
   * Fixed bug #859465 (Fatal error if a PUL contains two deactivate IC primitives)
   * Fixed bug #859467 (Fatal error if a PUL contains two activate Foreign Key primitives)

=== modified file 'include/zorba/static_context.h'
--- include/zorba/static_context.h	2011-11-15 08:23:20 +0000
+++ include/zorba/static_context.h	2011-11-17 05:05:28 +0000
@@ -586,7 +586,7 @@
    * @return the audit event
    */
   virtual audit::Event*
-  getAuditEvent() = 0;
+  getAuditEvent() const = 0;
 
 
   /** \brief Returns the QName of all external variables within the

=== modified file 'include/zorba/zorba.h'
--- include/zorba/zorba.h	2011-06-17 03:47:29 +0000
+++ include/zorba/zorba.h	2011-11-17 05:05:28 +0000
@@ -33,6 +33,7 @@
 #include <zorba/collection_manager.h>
 #include <zorba/xquery.h>
 #include <zorba/zorba_string.h>
+#include <zorba/iterator.h>
 
 namespace zorba {
 

=== modified file 'src/api/staticcontextimpl.cpp'
--- src/api/staticcontextimpl.cpp	2011-11-15 08:23:20 +0000
+++ src/api/staticcontextimpl.cpp	2011-11-17 05:05:28 +0000
@@ -1424,7 +1424,7 @@
 
 
 audit::Event*
-StaticContextImpl::getAuditEvent()
+StaticContextImpl::getAuditEvent() const
 {
   return theCtx->get_audit_event();
 }

=== modified file 'src/api/staticcontextimpl.h'
--- src/api/staticcontextimpl.h	2011-11-15 08:23:20 +0000
+++ src/api/staticcontextimpl.h	2011-11-17 05:05:28 +0000
@@ -246,7 +246,7 @@
   setAuditEvent(audit::Event* anEvent);
   
   virtual audit::Event*
-  getAuditEvent();
+  getAuditEvent() const;
 
   virtual void
   getExternalVariables(Iterator_t& aVarsIter) const;  

=== modified file 'src/context/dynamic_loader.cpp'
--- src/context/dynamic_loader.cpp	2011-10-19 16:19:45 +0000
+++ src/context/dynamic_loader.cpp	2011-11-17 05:05:28 +0000
@@ -191,14 +191,6 @@
 }
 
 
-DynamicLoader&
-DynamicLoader::getInstance()
-{
-  static DynamicLoader singleton;
-  return singleton;
-}
-
-
 DynamicLoader::~DynamicLoader()
 {
   for (LibrarySet_t::const_iterator lIter = theLibraries.begin();
@@ -265,7 +257,7 @@
 
       if (modfile->good())
       {
-        ExternalModule* lModule = getInstance().loadModule(potentialModuleFile);
+        ExternalModule* lModule = loadModule(potentialModuleFile);
         if (lModule)
         {
           if (lModule->getURI().c_str() != aNsURI)

=== modified file 'src/context/dynamic_loader.h'
--- src/context/dynamic_loader.h	2011-08-08 13:51:27 +0000
+++ src/context/dynamic_loader.h	2011-11-17 05:05:28 +0000
@@ -29,18 +29,17 @@
 class DynamicLoader
 {
 public:
-  static ExternalModule* getExternalModule(
+  ExternalModule* getExternalModule(
       zstring const& aNsURI,
       static_context& aSctx);
 
 private:
 
+  friend class GlobalEnvironment;
   DynamicLoader();
 
   ~DynamicLoader();
 
-  static DynamicLoader& getInstance();
-
   ExternalModule* loadModule(const zstring& aFile) const;
 
 #ifdef WIN32

=== modified file 'src/context/static_context.cpp'
--- src/context/static_context.cpp	2011-11-15 08:23:20 +0000
+++ src/context/static_context.cpp	2011-11-17 05:05:28 +0000
@@ -207,7 +207,7 @@
     if (dyn_loaded_module)
     {
       ZORBA_ASSERT(sctx);
-      module = DynamicLoader::getExternalModule(lURI, *sctx);
+      module = GENV_DYNAMIC_LOADER->getExternalModule(lURI, *sctx);
 
       // no way to get the module
       if (!module)
@@ -2634,7 +2634,7 @@
   // dynamic loader
   if (!found)
   {
-    lModule = DynamicLoader::getExternalModule(aURI, *this);
+    lModule = GENV_DYNAMIC_LOADER->getExternalModule(aURI, *this);
 
     // no way to get the module
     if (!lModule)
@@ -3400,7 +3400,7 @@
 /***************************************************************************//**
 
 ********************************************************************************/
-audit::Event* static_context::get_audit_event()
+audit::Event* static_context::get_audit_event() const
 {
   const static_context* sctx = this;
   audit::Event* res = sctx->theAuditEvent;

=== modified file 'src/context/static_context.h'
--- src/context/static_context.h	2011-11-15 08:23:20 +0000
+++ src/context/static_context.h	2011-11-17 05:05:28 +0000
@@ -917,7 +917,7 @@
   //
   void set_audit_event(audit::Event* ae);
 
-  audit::Event* get_audit_event();
+  audit::Event* get_audit_event() const;
 
 
   //

=== modified file 'src/system/globalenv.cpp'
--- src/system/globalenv.cpp	2011-11-02 06:14:25 +0000
+++ src/system/globalenv.cpp	2011-11-17 05:05:28 +0000
@@ -34,6 +34,7 @@
 #include "types/schema/schema.h"
 #include "context/root_static_context.h"
 #include "context/default_url_resolvers.h"
+#include "context/dynamic_loader.h"
 #include "functions/library.h"
 #include "annotations/annotations.h"
 #include "compiler/api/compiler_api.h"
@@ -98,6 +99,8 @@
   m_globalEnv->m_compilerSubSys = lSubSystem.release();
 
   m_globalEnv->m_http_resolver = new internal::HTTPURLResolver();
+
+  m_globalEnv->m_dynamic_loader = 0;
 }
 
 
@@ -105,6 +108,8 @@
 // note: destruction must be done in reverse initialization order
 void GlobalEnvironment::destroy()
 {
+  delete m_globalEnv->m_dynamic_loader;
+
   delete m_globalEnv->m_http_resolver;
 
   serialization::ClassSerializer::getInstance()->destroyArchiverForHardcodedObjects();
@@ -273,6 +278,15 @@
   return *m_compilerSubSys;
 }
 
+DynamicLoader* GlobalEnvironment::getDynamicLoader() const
+{
+  if (!m_dynamic_loader)
+  {
+    m_dynamic_loader = new DynamicLoader();
+  }
+  return m_dynamic_loader;
+}
+
 #ifdef ZORBA_XQUERYX
 XQueryXConvertor    *GlobalEnvironment::getXQueryXConvertor()
 {

=== modified file 'src/system/globalenv.h'
--- src/system/globalenv.h	2011-10-21 03:05:49 +0000
+++ src/system/globalenv.h	2011-11-17 05:05:28 +0000
@@ -27,6 +27,7 @@
 class RootTypeManager;
 class root_static_context;
 class XQueryXConvertor;
+class DynamicLoader;
 
 namespace internal {
 class HTTPURLResolver;
@@ -67,6 +68,8 @@
   internal::ThesaurusURLResolver  * m_thesaurus_resolver;
 #endif /* ZORBA_NO_FULL_TEXT */
 
+  mutable DynamicLoader  * m_dynamic_loader;
+
 public:
 
   static void init(store::Store* store);
@@ -108,6 +111,8 @@
   internal::ThesaurusURLResolver* getThesaurusURLResolver() const { return m_thesaurus_resolver; }
 #endif /* ZORBA_NO_FULL_TEXT */
 
+  DynamicLoader* getDynamicLoader() const;
+
 #ifdef ZORBA_XQUERYX
   XQueryXConvertor* getXQueryXConvertor();
 #endif
@@ -135,6 +140,8 @@
 
 #define GENV_ROOT_STATIC_CONTEXT GlobalEnvironment::getInstance().getRootStaticContext()
 
+#define GENV_DYNAMIC_LOADER GlobalEnvironment::getInstance().getDynamicLoader()
+
 }
 
 #endif /* ZORBA_GLOBALENV_H */