Code review comment for lp:~zorba-coders/zorba/structuralrelationships2

Revision history for this message
Federico Cavalieri (fcavalieri) wrote :

Hi Markos,
i noticed that the parse uri function now uses strtoul without checking errno with the following effects:

A missing component is considered 0:
np:in-same-tree-of(xs:anyURI("zorba:..1.50"),xs:anyURI("zorba:0.0.1.50")) ->true

Out of ulong value-space == ULONG_MAX
np:in-same-tree-of(xs:anyURI("zorba:.3333333333333333333333333333333333333333333333333333333333333333333333333333333333333333333333333333333333333.1.50"),xs:anyURI("zorba:0.222222222222222222222222222222222222222222222222222222222222222222222222222222222222222222222222222222222.1.50")) ->true

Is this ok?

Thanks
Federico

« Back to merge proposal