Skip to content
This repository has been archived by the owner on Mar 26, 2020. It is now read-only.

Adds Qt Qml compatible records #421

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

maPaydar
Copy link

@maPaydar maPaydar commented Dec 8, 2018

uses Qt basic types instead of std
adds Q_DECLARE_METATYPE to access objects from qml context

uses Qt basic types instead of std
adds Q_DECLARE_METATYPE to access objects from qml context
@paulocoutinhox
Copy link

Very nice :)

@xianwen
Copy link

xianwen commented Dec 26, 2018

Hi, @maPaydar:
I noticed you haven't signed the CLA yet, could you please sign it here: https://opensource.dropbox.com/cla/
Thanks a lot!

Copy link
Contributor

@artwyman artwyman left a comment

Choose a reason for hiding this comment

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

Can you add some examples and test cases, and check in the generated files, so reviewers can see what this actually generates?

@artwyman
Copy link
Contributor

artwyman commented Apr 9, 2019

This repo isn't going to be adding any new features so I'm putting this PR on hold.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants