Jody Garnett ha scritto:
> Andrea Aime wrote:
>> Two proposals:
>> * move StyleAttributeExtractor to rendering, there is
>>    actually nothing in gt2 using it besides rendering
>>   
> Agreed - move it.

Cool

>> * subclass both of those classes in renderer to take
>>    care of the dynamic symbols proposal
>>   
> Eventually I would like to move the style implementation into renderer 
> as well; nothing else in main makes use of it.

I believe the go renderer might use them? Instead of moving to
renderer, what about a gt-style module? Anyways, I'm not going to
make that change right now, I'm just talking.

>> I like the first one better, but I'm open to the second one as well? 
>> Opinions? If no one says anything I'll
>> perform the move soon (say, in two days, to allow for someone to reply?).
>>   
> 
> Quick question does the class need to be public? And can we document it 
> here:
> - 
> http://docs.codehaus.org/display/GEOTDOC/08+StyleVisitor+use+with+DuplicatingStyleVisitor+and+RescaleStyleVisitor
>  

Does the class need to be public? Yes, we use it in the shapefile
renderer as well, the GO renderer might be interested in using it too
(along with SLDStyleFactory and company, which would make it dependent
on the renderer module).

About documenting it, I believe that just pointing to the class is
sufficient, people can look the javadoc. I don't see the need of a
tutorial like docs for a class that's used only for optimization
purposes in the renderers?

Cheers
Andrea

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to