[udig-devel] Bleck; sorry that was more of a rant

[Andrea Antonello]

Hi Jody,

> I am afraid that last email was more of a rant; I am more upset that
> I failed to communicate one of the more interesting parts of the uDig
> design.

do not feel sorry about that. This is a developers list, I can expect
critics and am glad if someone redirects me a bit. If you ment to be
rude, my English knowledge didn't get this through to me like that
(lovely ignorance) :)

> The ShapeMover code was brave and one of the only examples to even
> use the IResolveAdapterFactory facilities. I will endevour to improve
> the documentation and code base so it is easier to use the future.

Yeah, honestly I have big big difficulties to understand some design
"perversions" (as long as they don't get clear to me :)). So if you
will be able to give some docs, I will study.

The shapemover is not in use now, but will be part of the consolidation
mechanism, which should move every local resource into the project
folder in order to be able to just move the folder to another
udig installation and be usable - workspace portability.

I will work on that at the European Code Sprint next Monday. I see you
are in Europe and very busy, but perhaps some of you guys could be in
IRC in those days to give advices and help?

Thanks,
Andrea


> 
> Jody
> > This class is both a factory( IResolveAdapterFactory) and a 
> > destination (literally the ShapeMover class creates itself). This
> > is really really confusing when first looked at!
> > - ShapeMover() no argument constructor is as a IResolveFactory in 
> > order to check canAdapt
> > - If canAdapt returns true the adapt method will end up calling new 
> > ShapeMover( IResolve )
> >
> > While that is a little bit of fun; the kicker for me is the check
> > in canAdapt:
> >> return adapter.isAssignableFrom(ShapeMover.class);
> > Can I be reading this right? We can adapt if we want to? That is
> > all the time; we never check the provided handle...
> >
> > Please tell me this code is not used...
> > Jody
> >
> >
> > _______________________________________________
> > User-friendly Desktop Internet GIS (uDig)
> > http://udig.refractions.net
> > http://lists.refractions.net/mailman/listinfo/udig-devel
> 
> _______________________________________________
> User-friendly Desktop Internet GIS (uDig)
> http://udig.refractions.net
> http://lists.refractions.net/mailman/listinfo/udig-devel