oh, the addition of ":" to separate the namespace from the localname in 
toString() (and thus in Name.getURI() ), breaks TypeSystemTest.testNaming()

Look at the following snipet:

 Name n1 = name("gopher://localhost/test/";, "this");
 assertEquals(name("gopher://localhost/";, "test/this"), n1);

it expects two names with different namespace and localname happening 
to "address" the same "node" to be equivalent...

now I wonder if that's correct. I know there are some issues regarding 
ISO19103 here, but I don't know where the missconception is.

From my (gml'ized) point of view, gopher://localhost/test and 
gopher://localhost are different namespaces, and test/this should not be 
acceptable as a "localName", as it is more a location path....

Without ":" separating namespace from localname in Name.getURI(), the test 
where working, but it seemed forced... as if I don't include the trailing 
slash in gopher://localhost/test/ and gopher://localhost/ the result is not 
going to be the same, and the trailing slash should be innocuous anyway...

My proposal (based on an ignorance of ISO19103 and why it is needed in Feature 
land) is to keep Name and TypeName as simple as possible, that is, namespace 
is namespace, and localName is localName, both together don't form a "path"?

any comment will be highly appreciated

Gabriel. 


On Wednesday 23 May 2007 11:45:33 Gabriel Roldán wrote:
> thanks Justin,
>
> done.
>
> Gabriel
>
> On Tuesday 22 May 2007 16:26:02 Justin Deoliveira wrote:
> > +1 from me :).
> >
> > Gabriel Roldán wrote:
> > > Hi,
> > >
> > > is it ok if I make Name and TypeName toString() to return
> > > namespace:localName instead of namespacelocalName?
> > > (note the colon separating ns from local name).
> > > It woud just make my life a bit easier :)
> > >
> > > Gabriel
> > >
> > > -----------------------------------------------------------------------
> > >-- This SF.net email is sponsored by DB2 Express
> > > Download DB2 Express C - the FREE version of DB2 express and take
> > > control of your XML. No limits. Just data. Click to get it now.
> > > http://sourceforge.net/powerbar/db2/
> > > _______________________________________________
> > > Geotools-devel mailing list
> > > [email protected]
> > > https://lists.sourceforge.net/lists/listinfo/geotools-devel
> > >
> > > !DSPAM:4007,4652c906322761015089218!
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> Geotools-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geotools-devel



-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to