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

Revision history for this message
Markos Zaharioudakis (markos-za) wrote :

> > > > function name too long? node-structural-information
> > >
> > > Any suggestions?
> > >
> > > node-information
> > > node-position
> > > node-uri
> > > structural-information
> > I like node-information as name for both - the module and the function.
> > That's at least consistent with the node-reference module and function.
>
> Fixed

I dislike the name "node-information" because it conveys no information about what the module is doing. I think "node-position", though not perfect, is the most accurate one. The comments in the .xq file speak of "node positions" as well.

« Back to merge proposal