Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added more gtk3 wrapped functions #720

Merged
merged 1 commit into from
Apr 21, 2023
Merged

Added more gtk3 wrapped functions #720

merged 1 commit into from
Apr 21, 2023

Conversation

zohanzephyr
Copy link
Contributor

No description provided.

@ptitSeb
Copy link
Owner

ptitSeb commented Apr 21, 2023

Thank you

@ptitSeb ptitSeb merged commit 1a3d412 into ptitSeb:main Apr 21, 2023
// unwrap (and use callback if not a native call anymore)
static void unwrapGApplicationClass(my_GApplicationClass_t* class)
{
#define GO(A, W) class->A = find_##A##_GApplication (class->A) //SUPERGO()全部定义,但并未全部调用
Copy link
Collaborator

Choose a reason for hiding this comment

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

Some Chinese here, probably change it to English?

Copy link
Owner

Choose a reason for hiding this comment

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

ah yeah, will do

Copy link
Collaborator

Choose a reason for hiding this comment

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

/home/ksco/Develop/box64/src/wrapped/wrappedgobject2.c:908:65: warning: backslash and newline separated by space
  908 |     SetGApplicationID(my->g_application_get_type());            \
      |                                                                  

Also a warning, need to remove trailing spaces.

Copy link
Owner

Choose a reason for hiding this comment

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

Yep, saw that too.

@zohanzephyr
Copy link
Contributor Author

Sorry I ignored Chinese

ptitSeb added a commit that referenced this pull request Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants