Fixing compilation issue on Ubuntu #335

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Collaborator

detro commented Oct 31, 2012

Reported here.
Conversion from NULL to QVariant() doesn't work
implicitly on Ubuntu.

@detro detro closed this Oct 31, 2012

Collaborator

detro commented Oct 31, 2012

On a rush I read the signature wrong.

Never code on a rush. NEVER EVER.

Fix compilation issue on Ubuntu.
Reported [here](402a8d9#commitcomment-2079231).
Conversion from `NULL` to `QVariant()` doesn't work implicitly on Ubuntu.

@detro detro reopened this Oct 31, 2012

Collaborator

detro commented Oct 31, 2012

OK, now it should be fine.
I missed that the 4th argument is a QString, not a QVariant.

Owner

ariya commented Nov 1, 2012

Landed. Thanks!

@ariya ariya closed this Nov 1, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment