Ghislain, is there any reason why inside initializeAsQNameNotInPool() you do:

  GET_STORE().getQNamePool().insert(aNamespace, aPrefix, aLocalName);

instead of

  GET_STORE().getQNamePool().insert(aNamespace, zstring(), aLocalName);

It seems to me that you are inserting an unnormalized qname for no good reason?

Also, the new ternary QNameItem constructors are not used anywhere in the 
simplestore; they are to be used by the sausalito store only, right? If so, can 
you add a comment to say so?
-- 
https://code.launchpad.net/~zorba-coders/zorba/qname-pool-refactoring/+merge/94953
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

Reply via email to