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

Request for review: cbProject::operator=

(1/3) > >>

oBFusCATed:
Here is the patch: http://cmpt.benbmp.org/codeblocks/patches/cbproject.copy.patch

Why I need this operator - because I need to fix this bug:
1. open project options
2. goto build targets
3. remove a target
4. close the dialog with cancel
5. reopen the dialog

What the user expects is that the target is still there, but in fact it is not, because the dialog works with the original object and not with a copy.
This is not a serious problem when using a modal dialog, but it is when the project options have been moved in an editor.

So any comments about the code will be welcome.

MortenMacFly:
Well first it does not compile. :-)

It errors at:

--- Code: ---        ProjectFile* pf = new ProjectFile(this, **it);

--- End code ---
...in cbproject.cpp.

Maybe I am missing something? (BTW: I was very happy to see that the patch applied seamlessly. ;-))

oBFusCATed:
What is the error?

MortenMacFly:
I can't try at the moment but if i recall correctly the constructor was missing.

oBFusCATed:
If you look at the patch it is there.

Navigation

[0] Message Index

[#] Next page