On 13/03/2025 15:02, Alexandra Wang wrote:
Hi Mark,
Thank you so much for reviewing! I have attached the new patches.
Hi Alex,
I am reviewing this from a feature perspective and not from a code
perspective. On the whole, this looks good to me from a standards point
of view.
There are two things that stand out to me about this feature:
1) I am not seeing the ** syntax in the standard, so does it need to be
signaled as not supported? Perhaps I am just overlooking something.
2) There is no support for <JSON item method>. Since this appears to be
constructing a jsonpath query, could that not be added to the syntax and
allow jsonpath to throw the error if the function doesn't exist?
--
Vik Fearing
