Would you be able to either switch this (with a null check) or use TextUtils.equals()?

Hi, thanks for looking into this! I can change no problem.

The comment actually made me curious, and I searched around which form would be better, but I couldn't find general advice.

My preference was to avoid a verbose null check. The TextUtils would add an import, slighly less straightforward than String.equals(other). There should be some disadvantage on "literal".equals() too, perhaps readability?

Yeah, it just feels a bit odd to me. The check is really "Is f.name equal to ...", but the other way, written to avoid the null check, turns it around, and that doesn't seem as nice.

I kind of wish it were in Kotlin so we could avoid the whole question, but 🤷

me too, but the tool is nonetheless very usefull to me so thank you!