Hi Petr, On Tue, Oct 22, 2024 at 4:48 AM Petr Pavlu <[email protected]> wrote: > > I had some minor comment about adjusting the name of function > symbol_print_versions() and possibly changing sym->name to 'char *' on > the v2 of the patch: > https://lore.kernel.org/all/[email protected]/ > Please have a look, it seems it felt through the cracks.
Sorry, I missed that somehow. I can split this into two functions to avoid confusion. Also I'm using const char * to make it obvious that the name shouldn't be modified by users of struct symbol. This does require a cast when freeing the string, which isn't ideal, but I feel it's overall not a terrible trade-off. Sami
