Skip to content

Commit

Permalink
(ckb) Fixed QStringList usage
Browse files Browse the repository at this point in the history
  • Loading branch information
ccMSC committed Aug 26, 2015
1 parent 1126860 commit 2eed29a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/ckb/ckbsettings.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ bool CkbSettings::containsGroup(const QString &group){
if(components.length() > 1){
// Find sub-group
SGroup group(*this, components[0]);
return containsGroup(components.mid(1).join('/'));
return containsGroup(QStringList(components.mid(1)).join('/'));
}
return childGroups().contains(group);
}
Expand Down

3 comments on commit 2eed29a

@mrueg
Copy link

@mrueg mrueg commented on 2eed29a Aug 26, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you planning to release 0.2.1.1 that includes this fix?

@ccMSC
Copy link
Owner Author

@ccMSC ccMSC commented on 2eed29a Aug 27, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was already in the master but I've now moved the v0.2.1 tag to this commit instead of the old one.

The behavior is identical; the only reason this is a separate commit is because it wasn't compiling on Ubuntu 12.04 and I didn't catch it before I pushed the version change.

@Hitmarv
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, I just got the K95 and Im using it with OSX. Are macros available on this driver?

Great driver by the way.

Thanks!

Please sign in to comment.