Sure, separate patch is OK if you do not envision any other problem with such a move.


Sent from Mail.ru app for iOS


Monday, 9 September 2019, 18:54 +0300 from gorcunov@gmail.com <gorcunov@gmail.com>:
On Mon, Sep 09, 2019 at 06:11:44PM +0300, Sergey Ostanevich wrote:
> Cyrill,
>
> Why not to move the map_direct_symbols to the module level, so that
> creation and
> fill of the table will happens only once and not at every call to the
> output handler?

Well, initially I thought to keep such things inside serializer helper
so that if we need we will easily extend this table and the change
will be inside function context. I must admit I think all this comes
from a strong habbit of working with compilers where such thing would
be simply optimized. That said I agree we should do so.

I think this issue could be addressed on top as a separate patch?
Just to not resend the whole series.