How do I create a default value for a QVariantMap?

[mrjj]

Hi
You mean like an empty map if none is given ?
void SomeFunc( QWidget* parent, QVariantMap& Mymap = QVariantMap() ); // in.h

[graniteDev]

That compiled, but I got 10 warnings, and we can't make pull requests on code with warnings.

C4239: nonstandard extension used : 'default argument' : conversion from 'QMap<QString,QVariant>' to 'QVariantMap &'
A non-const reference may only be bound to an lvalue

C4239: nonstandard extension used : 'argument' : conversion from 'QMap<QString,QVariant>' to 'QVariantMap &'
A non-const reference may only be bound to an lvalue

C4239: nonstandard extension used : 'default argument' : conversion from 'QMap<QString,QVariant>' to 'QVariantMap &'
A non-const reference may only be bound to an lvalue

and another 7 more if your interested, all saying about the same thing.

[Christian Ehrlicher]

@graniteDev said in How do I create a default value for a QVariantMap?:

QVariantMap &AppMap

It must be 'const QVariantMap &AppMap' ...

[mrjj]

hmm
did you add const to it ?
i got no such warning from small test. (visual stud compiler 2015)

[graniteDev]

@mrjj Ok const fixed it. Thank you!

[mrjj]

@graniteDev
sorry my bad :) apparently visual stud compiled it anyway ..

[SGaist]

Hi,

Out of curiosity, why make it the second argument ? It's more common to have the parent as last parameter.

[graniteDev]

@SGaist No reason, the parent was already there before I implemented this, so I just typed after.

[Christian Ehrlicher]

@mrjj said in How do I create a default value for a QVariantMap?:

apparently visual stud compiled it anyway ..

Yes, this is a bug in MSVC which exists for a long time. Maybe they can't fix it due to backward compatibility reasons.

[graniteDev]

@SGaist This morning, turns out, a great reason arose, I don't need to overload the constructor if I put the arguments after the parent, since If I don't want to pass in a QVariantMap, I can leave it off and still provide a parent.