Allow allocated file masks for shell completions

[ozwaldorf]

As the title says, it would be useful to allow both borrowed and allocated types for a file mask.

Without looking at the code, my assumption is this should be fairly easy to accomplish just by replacing the field mask: &'static str with mask: Cow<str> and internally lower it to an &str. Library users would have to make a breaking change to add an into() call to their static masks

Example usecase

For lutgen, it would be nice to keep a static slice of file extensions directly, used for both creating a custom footer doc and the file mask. ATM, I have to define the mask itself (ie, *.png|*.jpg) and then for the doc remove *. and split the string along | chars to be able to use doc.literal() for each item. With this change, I can move my constant to just a slice, and construct the mask at runtime

[pacak]

One possible workaround is that you can allocate a String and then leak it. If you are not doing it in a loop leaking is safe. I'm planning to make something more flexible and drop the whole enum in favor of having several methods so I can pass mask: impl Into<String>, but that's some time away - rustdoc is a bit limiting and figuring out a few other things. I don't really want to make a breaking change just to break it again.

[ozwaldorf]

Sounds like a better api design solution 👍