I've changed to use zorba::store::StoreConsts::NsScoping.

Regarding vector vs. map, after thinking about it a bit, I agree with you that 
map<> would be at least a bit better user experience. However, 
ItemFactory::createElementNode() takes a vector<pair<>> already, and that can't 
be changed. I think probably it's best that getNamespaceBindings() be 
consistent with that API.

In fact, probably createElementNode() should be changed to use the same typedef 
as getNamespaceBindings(). Should I move the typedef for NsBindings into 
store_consts.h?
-- 
https://code.launchpad.net/~zorba-coders/zorba/get-namespace-bindings/+merge/87187
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