Petteri Hintsanen <[email protected]> writes:

> 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

That's exactly what I had in mind. Thank you.

-- 
   "Cut your own wood and it will warm you twice"

Reply via email to