search

sshyran / genxdm

Automatically exported from code.google.com/p/genxdm
0 stars 0 forks source link

ValidatorFactory (in processor.w3c.xs.validation) should not have an <N> parameter, and should not take TypedContext as an argument in its constructor #141

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
Stale code, clearly.

The constructor takes TypedContext<N, A>, and the class is parameterized on <N, 
A>. N is never used. The TypedContext is discarded after retrieving the 
AtomBridge<A> from it.

Solution: change the argument to the constructor to be AtomBridge<A>, and drop 
the N parameter from the class.

Possibly: *add* a second constructor, without removing the parameterization. 
This will avoid breaking things. But it really needs to be changed (breaking 
things) before 1.0, because it doesn't make sense to push out code that's 
brainless in 1.0. So at least @deprecate the old constructor.

Original issue reported on code.google.com by aale...@gmail.com on 26 Sep 2013 at 6:28

GoogleCodeExporter commented 8 years ago 
Defer

Original comment by aale...@gmail.com on 24 Oct 2013 at 5:27