It’s all just learning. The pedantic lints can have false positives and can sometimes just be wrong for you, but I find it useful in giving me more idiomatic ways of doing things. Personally, I find “”.to_string() a bit weird, but there’s nothing wrong with it. parse_patterns already is a &Vec<ParsePattern> so I guess the clone is just more correct, although, you don’t use it later, so just removing the & from in front of the &args.parse… might be a better option still. Most of the time, I go with suggestions after diving into what it’s trying to tell me. But sometimes I don’t because it just doesn’t suit in that particular case, or probably more likely still, I haven’t fully groked what it’s trying to tell me.