Hi Khaled, Thanks for looking at this and correcting the implementation.
I've verified the changes you've committed and am happy with the functional correctness of the fixes. Many thanks. On Wed, Jan 19, 2011 at 12:30 AM, Khaled Noaman <[email protected]> wrote: > > Hi Mukul, > > I took a look at your change. I noticed you were creating a new attribute > declaration every time an xsi attribute is referenced and setting the > enclosing parent property (attribute declarations with a global scope have > no parent). In SchemaGrammar class, we already have a static grammar > instance (1.0) representing the xsi namespace (containing declarations for > the 4 known xsi attributes), so we could use a similar mechanism and have it > work the same way built-in types are referenced. I have made the necessary > changes and checked in the code. > > Regards, > Khaled > -- Regards, Mukul Gandhi
