João Sousa: yes, I think preserving the case would be the right thing. But actually there is already an implementation for this in MP4 and ID3
Check the code for `cademap`. It stores the original casing on loading and uses it again on saving.
That means please carefully investigate where the actual problem lies
I suspect that it is not working for tags with defined mapping, like label, and only applies to freeform tags unknown to picard
Thinking about this there could also be an argument made for changing the casing for defined tags. Not fully sure. Do we maybe need an option?
Reading case insensitive definitely makes sense, though
Protopia[m] has quit
stevilknevil[m] has quit
JooSousa[m]
<outsidecontext[m> "Check the code for `cademap`. It..." <- Correct, the code does store the case in __casemap when loading, however when saving, if it is a label already mapped it saves to the mapped case. In the solution im working on, a label is checked if it exists in __casemap before the "default mappings" checks occur
outsidecontext[m
Just make sure this only happens for tags that are stored as free text tags
JooSousa[m]
<outsidecontext[m> "Just make sure this only happens..." <- you mean freeform tags?
