[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 130045] (Severe) Metadata editor: Clicking "Per Track" checkbox
From:       Mats Ahlgren <ahlgren () mit ! edu>
Date:       2006-06-30 14:32:38
Message-ID: 20060630143238.5412.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=130045         




------- Additional Comments From ahlgren mit edu  2006-06-30 16:32 -------
Nice work on the fix and discovery of the similar bug with discNumber.

However, this won't prevent the same problem from happening again should new tags \
become supported. It might be alot simpler and cleaner to define a list of all used \
tags/widget objects, and use that list throughout tagdialog.cpp. This would allow one \
to "clear" these objects with one line of code (map/for), and prevent similar errors \
in other places by working with the list instead of the individual items.

Alternatively, at the very least, it should be noted somewhere that this section of \
the code must be updated when new tags become supported, to prevent this from \
happening again.


[prev in list] [next in list] [prev in thread] [next in thread]