Hi, Thank you for the suggestion. I have created a PR with the required changes [1].
[1] https://github.com/apache/vxquery/pull/100 Thank you. Yours sincerely, Riyafa On 15 July 2016 at 22:10, Till Westmann <[email protected]> wrote: > Hi Riyafa, > > I think that it’s fine to add a new value for the Object union to the > ASTTag. > The grammar in the specification describes the syntax of the language, but > it > is up to the implementation to decide how to represent this internally. > > Please go ahead and create that tag for the new AST node. > > Cheers, > Till > > On 15 Jul 2016, at 8:24, Riyafa Abdul Hameed wrote: > > Hi, >> >> On attempting to create a separate constructor for SimpleObjectUnion I >> have >> come across the following issues: >> >> >> - Since there's no separate ValueTag for this SimpleObjectUnion >> constructor, it is not possible to call a separate translate method >> when >> this constructor is encountered. >> - A new ValueTag for the constructor cannot be created since the >> specification[1] doesn't mention about such a constructor. >> >> Am I to proceed creating this new SimpleObjectUnion constructor? If so any >> suggestions on how it could be done? >> >> [1] >> >> http://jsoniq.org/docs/JSONiqExtensionToXQuery/html-single/index.html#section-json-value-construction >> >> Thank you. >> >> Yours sincerely, >> >> Riyafa >> >> -- >> Riyafa Abdul Hameed >> Undergraduate, University of Moratuwa >> >> Email: [email protected] >> Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/> >> <http://facebook.com/riyafa.ahf> <http://lk.linkedin.com/in/riyafa> >> <http://twitter.com/Riyafa1> >> > -- Riyafa Abdul Hameed Undergraduate, University of Moratuwa Email: [email protected] Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/> <http://facebook.com/riyafa.ahf> <http://lk.linkedin.com/in/riyafa> <http://twitter.com/Riyafa1>
