panel: Add outputs option for per-output configuration

[soreau]

This adds a new option named 'outputs' that is a list of output names on which a panel instance should be rendered. The default value is '*' wildcard, which means all outputs.

[ammen99]

Having this dynamically allocated seems really weird, if you need to load an option at runtime, you can use .load_option(<option name>)

[soreau]

Sounds like this might work.

[soreau]

Well this sounded good but I was not able to get load_option() working to help here. Maybe I am using it wrong.

[trigg]

Sorry, I've read through it.

I currently don't know where I stand on the conversation but wonder why you're specifically avoiding doing this in shell-app.

[soreau]

... but wonder why you're specifically avoiding doing this in shell-app.

@trigg Thanks for looking. It is not the case that I set out with the intent to only touch panel.cpp, it's just the way the patch turned out. After I had it working, I noticed the only thing I had in wf-shell-app.hpp was moving the monitors list from private to public, and I never used it, in favor of the more convenient std::map that I'd installed.

I will have another look to see if using the existing monitors list is doable and get rid of the extra std::map though, if that's what you're expecting.

[soreau]

Before suggested changes: 2 files changed, 102 insertions(+), 2 deletions(-)
After suggested changes: 5 files changed, 98 insertions(+), 2 deletions(-)

Ca ne fait rien. 😀

EDIT: Adjusted to use std::find_if instead of a loop for checking if panel exists.
After suggested changes: 5 files changed, 94 insertions(+), 2 deletions(-)

[trigg]

My read through looks like the loops can be merged. Otherwise very happy with this, and if you're sure of yourself just merge this :)