On Jan 29, 2008 9:15 PM, Martin Desruisseaux
<[EMAIL PROTECTED]> wrote:
> Hello Simone
>
> I noticed the following in the Hints class:
>
>   String VALUE_OVERVIEW_POLICY_QUALITY = "Quality";
>   String VALUE_OVERVIEW_POLICY_IGNORE  = "Ignore";
>   String VALUE_OVERVIEW_POLICY_NEAREST = "Nearest";
>   String VALUE_OVERVIEW_POLICY_SPEED   = "Speed";
>
> They are not keys (like all other Hints constant declarations) but values for
> the Hints.OVERVIEW_POLICY key. Do you mind if I move those values in an
> OverviewPolicy enum? I would deprecated (not yet remove) the current ones for
> transition.
>

These hints were introduce by aaime. The only concern I could have
against turning them into an enum draws from the desire to backport to
2.4.x.
If we use an enum things might get a bit complicated. I am curious
about aaime's opinion here...

> Also can I replace (with deprecation cycle):
>
>   Key REPLACE_NON_GEOPHYSICS_VIEW = new Key(Boolean.class);
>
> by
>
>   Key COVERAGE_PROCESSING_VIEW = new 
> Key("org.geotools.coverage.grid.ViewType");
>
> ? The implication for users would be to replace:
>
>     hints.put(Hints.REPLACE_NON_GEOPHYSICS_VIEW, Boolean.TRUE);
>
> by
>
>     hints.put(Hints.COVERAGE_PROCESSING_VIEW, ViewType.PHOTOGRAPHIC);
>

+1

> Actually I have those changes ready on my local copy - waiting for your okay.
>
>         Martin
>

Simone.

-- 
-------------------------------------------------------
Eng. Simone Giannecchini
President /CEO GeoSolutions S.A.S.
Via Carignoni 51
55041  Camaiore (LU)
Italy

phone: +39 0584983027
fax:      +39 0584983027
mob:    +39 333 8128928


http://www.geo-solutions.it

-------------------------------------------------------

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to