[tarantool-patches] Re: [PATCH v2 3/8] json: lexer_eof and token_cmp helper functions

Konstantin Osipov kostja.osipov at gmail.com
Sat Oct 5 16:50:14 MSK 2019


* Vladislav Shpilevoy <v.shpilevoy at tarantool.org> [19/09/28 09:11]:
> >>  /**
> >> - * Compare JSON token keys.
> >> + * Compare JSON tokens as nodes of a JSON tree. That is, including
> >> + * parent references.
> >>   */
> > 
> > Please explain how it is useful. Why not json_tree_node_cmp then?
> 
> What is useful? Being able to compare two JSON nodes? Not sure if I
> understand the question. I just need to compare two nodes. In several
> places. And I added a function for that.

How it is independently useful , given there still exists the old cmp function.
You added a (yet) another cmp function, the comment should contain
the usage scenario for it.

-- 
Konstantin Osipov, Moscow, Russia




More information about the Tarantool-patches mailing list