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
2025-03-16 07514, 2025
outsidecontext[m
Check the code for `cademap`. It stores the original casing on loading and uses it again on saving.
2025-03-16 07538, 2025
outsidecontext[m
That means please carefully investigate where the actual problem lies
2025-03-16 07551, 2025
outsidecontext[m
I suspect that it is not working for tags with defined mapping, like label, and only applies to freeform tags unknown to picard
2025-03-16 07550, 2025
outsidecontext[m
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?
2025-03-16 07500, 2025
outsidecontext[m
Reading case insensitive definitely makes sense, though
2025-03-16 07524, 2025
Protopia[m] has quit
2025-03-16 07558, 2025
stevilknevil[m] has quit
2025-03-16 07540, 2025
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
2025-03-16 07529, 2025
outsidecontext[m
Just make sure this only happens for tags that are stored as free text tags
2025-03-16 07522, 2025
JooSousa[m]
<outsidecontext[m> "Just make sure this only happens..." <- you mean freeform tags?