[Release] ItemUpgrade.cpp ohne Warnungen

[Velmore]

Da bei vielen Warnungen kommen hier mal eine ItemUpgrade.cpp die keine Fehler gibt.



Hab da etwas noch geändert für das Fashion Kombine also die Sachen müsst ihr dann wieder rausnehmen oder die defines ändern.

PS: Da es eine .cpp Datei ist gibts kein VT.

[420twisted1]

can you explain better? I have no warnings in my ItemUpgrade.cpp

[Velmore]

Originally Posted by 420twisted1

can you explain better? I have no warnings in my ItemUpgrade.cpp

Some people gets warnings from the ItemUpgrade.cpp


In this there is the funktion for fashion combine to. Normaly this function is in dpsrvr.cpp

[jeromerz]

i think the mismatch something is causing that.. i have an error with that DWORD To BYTE something. but it doesn't effect the building or compiling process.

[Velmore]

Originally Posted by jeromerz

i think the mismatch something is causing that.. i have an error with that DWORD To BYTE something. but it doesn't effect the building or compiling process.

but some people think this is bad for there server.

[jeromerz]

maybe for some any way thanks for sharing ^^

[Wanetrain]

Die Warnungen entstehen durch dämliches Coding, was ja Combine ist.

Generel wenn du Warnungen Fixxen willst: zieh dir VS2012 Source, Compile den mal, 6.000+ Warnungen, hefe fun! :3

[Velmore]

Originally Posted by Wanetrain

Die Warnungen entstehen durch dämliches Coding, was ja Combine ist.

Generel wenn du Warnungen Fixxen willst: zieh dir VS2012 Source, Compile den mal, 6.000+ Warnungen, hefe fun! :3

Ja wegen den i dingern da. Aber unter 2003 sollte das keinen fehler geben, habs nicht mehr getestet da ich kein 2003 drauf habe.

[Wanetrain]

Originally Posted by Velmore

Ja wegen den i dingern da. Aber unter 2003 sollte das keinen fehler geben, habs nicht mehr getestet da ich kein 2003 drauf habe.

Na eigentlich weniger wegen den "i" dingern, sondern mehr weil so ziemlich alles Unsafe ist.

Trotzdem kleiner Tipp: überlege bei jedem Warning Fixx, was du machst!

[Velmore]

Originally Posted by Wanetrain

Na eigentlich weniger wegen den "i" dingern, sondern mehr weil so ziemlich alles Unsafe ist.

Trotzdem kleiner Tipp: überlege bei jedem Warning Fixx, was du machst!

Mir gings nur darum die Warnings in der ItemUpgrade.cpp weg zu machen.

Unter 2003 waren das ja die einzigen warnungen.

[Мentus]

Sind das nicht die Warnungen die entstehen, weil bei irgendwelchen Funktionen ein DWORD anstatt BYTE übergeben wird?

Da könnte man theoretisch static_cast< BYTE > nutzen oder einfach (BYTE) .. oder einfach die Variable mit einem unsigned char bzw BYTE definieren anstatt mit deinem DWORD... was eh unnötig ist.

[Wanetrain]

Originally Posted by Мentus

Sind das nicht die Warnungen die entstehen, weil bei irgendwelchen Funktionen ein DWORD anstatt BYTE übergeben wird?

Da könnte man theoretisch static_cast< BYTE > nutzen oder einfach (BYTE) .. oder einfach die Variable mit einem unsigned char bzw BYTE definieren anstatt mit deinem DWORD... was eh unnötig ist.

static_cast dient eigentlich nur zur Typ Convertierung, was also nicht dort zu verwenden ist.

Ach, noch was: es ist natürlich egal wenn man dort ein BYTE hat, allerdings ein int oder sonst was einspeist, nicht wegen einem Overflow (was bei einem BYTE nicht geht) sondern mehr wegen Memory Leaks, was FlyFF verdammt viele hat, Intel XE laden regelt, jungs.

Wie gesagt, wenn man kA von dem Zeug hat sollte man es lassen, oder sich zumindest AUSGIBIG darüber Infos einholen, danke.

[Мentus]

Originally Posted by Wanetrain

static_cast dient eigentlich nur zur Typ Convertierung, was also nicht dort zu verwenden ist.

Ach, noch was: es ist natürlich egal wenn man dort ein BYTE hat, allerdings ein int oder sonst was einspeist, nicht wegen einem Overflow (was bei einem BYTE nicht geht) sondern mehr wegen Memory Leaks, was FlyFF verdammt viele hat, Intel XE laden regelt, jungs.

Wie gesagt, wenn man kA von dem Zeug hat sollte man es lassen, oder sich zumindest AUSGIBIG darüber Infos einholen, danke.

Wirr wollen den Typ konvertieren ( umwandeln )sonst wird der Warning ja weiterhin ausgegeben.
Aber wie gesagt, der beste Weg ist einfach den lächerlichen DWORD in ein BYTE zu ändern. Da die ObjId eh nie über 255 hinaus geht.

.. und bei den ganzen < > Warnings, liegts meist an den for Schleifen bzw an .size() eines Vectors, der returnt ein size_t( size_type ) und die meisten nutzen einfach n int und schon spuckt er n Warning aus.