I commited as of revision 22110 a CommonFactoryFinder class that may be used as 
a replacement of StyleFactoryFinder and FilterFactoryFinder (I have not yet 
replaced the references to those factories finder however). Peoples asked for 
constructor injection. Do we have a volunter for adding the appropriate 
constructor to the relevant class, and send me the name of modified classes 
please? I mean, if a class is like that:

    class Foo() {
        public Foo() {
        }
    }

I would like a volunter for writting:

    class Foo() {
        private MyFactory mf;

        /**
         * @deprecated Replaced by [EMAIL PROTECTED] #Foo(Hints)}
         */
        public Foo() {
        }

        /**
         * @since 2.4
         */
        public Foo(MyFactory mf) {
            this.mf = mf;
        }
    }

and let me know which classes were updated that way, so I can add the following 
constructor (which may involve extending the CommonFactoryFinder class):

         public Foo(Hints hints) {
             this(CommonFactoryFinder.getMyFactory(hints));
         }

I this context, FactoryFinders are merely a way to provides default value to 
constructor injection in a configurable way.

     Martin

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to