ist nicht genug ?

[re_Look]

statt
+#ifdef FB_USE_PALETTE
+ int dxx;
+ if (dx < 0) {
+ fprintf(stderr, "ERROR: CFrameBuffer::paintBoxRel called with dx < 0 (%d)\n", dx);
+ dxx = 0;
+ } else
+ dxx = dx;
+#endif
das ?
+#ifdef FB_USE_PALETTE
+ if (dx < 0) {
+ fprintf(stderr, "ERROR: CFrameBuffer::paintBoxRel called with dx < 0 (%d)\n", dx);
+ dx = 0;
+ }
+#endif

[seife]

Probiere es doch einfach
Ein Blick auf den Prototype von CFrameBuffer::paintBoxRel() könnte auch helfen.

[flasher]

Um die Unlesbarkeit zu fördern kannst Du auch:

dx = (dx < 0) ? 0 : dx;

schreiben.
Das würde nur in der Hinsicht problematisch werden, da Du eine const var verändern möchtest. (call will abort at runtime, o.ä)
Daher auch der Hinweis auf den Prototyp

Gruß

[re_Look]

Achso const hab ich übersehen, dann reich aber:
memset(pos, col, ((dx < 0) ? 0 : dx));

[seife]

Das ist vermutlich langsamer (ich bin kein PPC-Assemblerspezialist, deswegen "vermutlich").
Und die Warnung wird ja nicht zum Spaß ausgegeben: da ist ein Bug im Code.

[rhabarber1848]

* bump *

Gibt es hier noch was zu tun?

[seife]

IMHO nicht, der Code wie er jetzt im CVS ist, ist korrekt.