Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

small clean up in cbeditor

(1/1)

frithjofh:
hi everyone,

i am proposing a small code change in the area of cbeditor.

i removed a variable in editorbase which was used to hold the information whether a given pointer refers to a builtin editor or not. i replaced it with the proper override in the derived class.

reasoning: 1) breaks nothing 2) expresses more clearly the relationship 3) removes several lines of code

patch file attached.

regards

oBFusCATed:
1 is clearly not true, because you've broken the ABI, so you need to rise the sdk version numbers in the cbplugin.h (just the last number).
Also virtual functions are a bit slower than normal methods. I doubt it would matter too much, but still.

killerbot:
patch seems ok

Navigation

[0] Message Index