In computing the value of the is_element_test variable, why do we need the 
condition:

dynamic_cast<const NodeXQType*>(m_content_type.getp())->m_schema_test == false

? In fact, this seems wrong to me, because if "this" is 
document-node(SchemaElementTest), then we will return true without really 
checking the SchemaElementTest.

BTW, the dynamic_cast is not necessary since you have already checked the type 
kind. A static_cast is better here.
-- 
https://code.launchpad.net/~nbrinza/zorba/bugs/+merge/81480
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