On 27.7.2023 18.30, Yoni Rabkin wrote:
As for the rest, can you please split it out to a separate file such as
"emms-info-native-spc" or "emms-info-native-id666" or similar? That will
scale better as we add more native info methods.

To simplify things, I am planning to split emms-info-native to separate files and namespaces, e.g.:

  emms-info-mp3
  emms-info-vorbis
  emms-info-opus
  emms-info-flac

So I'd recommend to use just "emms-info-id666" or "emms-info-spc" for this new addition.

In a hindsight, it was a bad choice to include implementation detail ("native") into a name. Well, naming tends to be hard.

thanks,
Petteri


Reply via email to