Possibly yes. I'll take a look
Garrett Rooney wrote:
> [snip]
> Nice work James! A few comments:
>
>> + void setElementWrapper(Element element, Element wrapper);
>
> Are we sure this should be part of the Factory interface? It looks
> like every place it's used we've already cast to FOMFactory, so could
> it just live there? I'm not sure if a new parser back end would keep
> track of element wrappers in the same way.
>
>> +public class InReplyToImpl
>> + extends ElementWrapper
>> + implements InReplyTo {
>
> Does it still make sense to have separate Impl classes and interfaces
> for the thread code?
>
> -garrett
>
