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

更改了xmake工程的配置 #5

Merged
merged 8 commits into from
Feb 27, 2023
Merged

更改了xmake工程的配置 #5

merged 8 commits into from
Feb 27, 2023

Conversation

linshire
Copy link
Contributor

No description provided.

Copy link
Collaborator

@BernardXiong BernardXiong left a comment

Choose a reason for hiding this comment

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

加油

include/stdarg.h Outdated
@@ -1,7 +1,7 @@
#ifndef __STDARG_H__
#define __STDARG_H__

typedef char * va_list;
#define va_list; __builtin_va_list
Copy link
Collaborator

Choose a reason for hiding this comment

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

这里正确?va_list;

@@ -1,7 +1,7 @@
#ifndef __STDARG_H__
#define __STDARG_H__

typedef char * va_list;
#define va_list __builtin_va_list
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个代码对齐也太糟糕了

Copy link
Contributor Author

Choose a reason for hiding this comment

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

在本地是正确的,已经修改过了


#define NULL 0

typedef unsigned char uint8_t;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why for arm? These should be the common definitions.

include/stdarg.h Outdated
#define va_start(v,l) __builtin_va_start(v,l)
#define va_end(v) __builtin_va_end(v)
#define va_end(v) __builtin_va_end(v)
Copy link
Collaborator

Choose a reason for hiding this comment

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

看看网页版本,乱的。另外,你这边是TAB

@BernardXiong BernardXiong merged commit adba279 into plctlab:main Feb 27, 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.

2 participants