KeXXeN hat geschrieben:@mws
Weil es wie eine öffentliche Abmahnung rüber kam
es war keine abmahnung, sondern ein revert von quatsch.
eine abmahnung wäre gewesen, wenn man ihm androht den cvs zugang zu entziehen. das ist nicht meine absicht.
ich habe lediglich darauf hingewiesen, das die TRY & ERROR methode nicht immer die richtige ist, sondern man sich komplett mit allen faktoren die bei so einer sache die rolle spielen, auseinander setzen muss. *klarstell*
das äussert sich nun mal in einem satz wie
revert metallicas new/delete fix
Metallica - what should happen in your opinion, if you construct an object
via new within the ctor and at the end of ctor you are deleting it AND
it still points to some memory?
that pig is USED within e.g. the paint_pig function
think about _what_ you are doing instead of fixing things that you do not
understand correctly.
nichts anderes sagt dieser satz aus - der infoviewer wurde konstruiert, ein pig object erzeugt, und das sofort wieder gelöscht!
dadurch ist zwar ein memleak weg, das aber nicht existieren würde, wenn man nachsieht, warum der dtor nicht aufgerufen wird.
solange wie kein anderer prozess mem allokiert, kann das sogar gutgehen. NUR keiner wird sich, wenn es dann mal passiert,
das der infoviewer abstürzt (sigsegv - signal 11) an diesen FALSCHEN fix erinnern.
desweiteren wäre dieser bug sehr schwer zu reproduzieren, weil
man den infoviewer öffnen muss, und auch drauf warten, das genau der speicher den man dort benutzt hat, wieder benutzt wird.
fazit:
hier wären jede menge user am weinen, das sporadisch wenn man epg anzeigen lässt, der dialog nach unterschiedlichen zeiten nicht mehr gemalt wird und neutrino crashed.
wollt ihr lieber bugs, oder nehmt ihr auch mal kritik an, wenn jemand was - vielleicht auch unbewusst - falsch macht?
letzteres spart euch und anderen nerven, denkt mal drüber nach.
gruss
mws
p.s.: bei alexw waren es damals gründe die richtig waren, man kann eine API halt nicht einfach so nach eigenem gedünken abändern. das alexw deswegen
die flinte ins korn wirft, ist eher bei ihm zu suchen, als bei obi oder dem tuxbox project ansich.