I will try to clean up some weird pre-existing QByteArray stuff (you marked it as "deep copy ?").
Done! Xml class cleaned up greatly.
I think the reason that messy QByteArray stuff was in there was that in the long journey to switch
to Qt4 which was really a major Qt change, these things were added to make it 'just work',
rather verbosely without looking deeply at the code.
It appears to be nice and streamlined now. Should hopefully be OK.
Tested OK with several song files, and re-saving them. Stepping through with debug to check as well.
(Funny, the older I get, the more I can enact sweeping changes in just a few hours that perplexed me for long times
ages ago. Qt4 really was a major change with its QString and QByteArray, sources of much confusion)
Try 'er out. Lemme know if any trouble with xml files such as song files and stuff.
In MusE git master now. ChangeLog:
- Xml class full makeover.
Removed all QByteArray stuff. Now it works directly with _s1 or _s2,
which cures concerns about deep copying the b.a. back to _s1/_s2,
stuff which was hastily added at Qt4 to make it work.
Tightened up several of the routines.