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

Revision history for this message
Matthias Brantner (matthias-brantner) 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.
I'm fine with node-position.

« Back to merge proposal