[Tarantool-patches] [PATCH v12 3/8] module_cache: improve naming

Vladislav Shpilevoy v.shpilevoy at tarantool.org
Sun Jan 24 19:27:12 MSK 2021


Thanks for the patch!

Very dubious commit. To me the old and new names look the same.
'func_name' was even better than 'func_name_desc'. At least
because it was shorter, and 'desc' does not add any more meaning.

On 18.01.2021 21:35, Cyrill Gorcunov via Tarantool-patches wrote:
>  - rename func_name to func_name_desc because
>    it is not just a name but rather a name descriptor
>    which includes symbol address

I don't see any address in it. Only name tokens.

>  - rename func_split_name to parse_func_name because
>    the action of splitting name to parts is not what
>    the function about, the function parses specification
>    to fetch a descriptor
> 
> Part-of #4642
> 
> Signed-off-by: Cyrill Gorcunov <gorcunov at gmail.com>
> ---


More information about the Tarantool-patches mailing list