> On 18 May 2021, at 15:50, Heikki Linnakangas <hlinnaka@iki.fi> wrote:
> The reason is that if you specify multiple --rmgr options, only the last one takes effect.
That's in line with how options are handled for most binaries, so this will go
against that. That being said, I don't think thats a problem here really given
what this tool is and it's intended usecase.
This patch makes the special case "--rmgr=list" a bit more awkward than before
IMO, as it breaks the list processing, but nothing we can't live with.
> I propose the attached to allow selecting multiple rmgrs
I agree with the other +1's in this thread, and am marking this as ready for
committer.
As a tiny nitpick for readability, I would move this line inside the string
comparison case where the rmgr is selected. Not that it makes any difference
in practice, but since that's where the filtering is set it seems a hair
tidier.
+ config.filter_by_rmgr_enabled = true;
--
Daniel Gustafsson https://vmware.com/