-
-
Notifications
You must be signed in to change notification settings - Fork 387
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
Experimental iOS Support #8407
base: master
Are you sure you want to change the base?
Experimental iOS Support #8407
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clang-Tidy
found issue(s) with the introduced code (1/2)
@@ -23,6 +23,8 @@ | |||
#include <ostream> | |||
#include <stdexcept> | |||
|
|||
#include <CoreFoundation/CoreFoundation.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CoreFoundation/CoreFoundation.h
file not found
*/ | ||
|
||
/* Include the SDL main definition header */ | ||
#include "SDL_main.h" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SDL_main.h
file not found
#define NS_ENUM(_type, _name) enum _name : _type _name; enum _name : _type | ||
typedef metal::int32_t EnumBackingType; | ||
#else | ||
#import <Foundation/Foundation.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#import is a language extension
#define NS_ENUM(_type, _name) enum _name : _type _name; enum _name : _type | ||
typedef metal::int32_t EnumBackingType; | ||
#else | ||
#import <Foundation/Foundation.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Foundation/Foundation.h
file not found
typedef metal::int32_t EnumBackingType; | ||
#else | ||
#import <Foundation/Foundation.h> | ||
typedef NSInteger EnumBackingType; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use using
instead of typedef
typedef NSInteger EnumBackingType; | |
using EnumBackingType = int; |
|
||
#include <simd/simd.h> | ||
|
||
typedef NS_ENUM(EnumBackingType, BufferIndex) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all parameters should be named in a function
typedef NS_ENUM(EnumBackingType, BufferIndex) | |
typedef NS_ENUM(EnumBackingType /*unused*/, BufferIndex /*unused*/) |
BufferIndexUniforms = 2 | ||
}; | ||
|
||
typedef NS_ENUM(EnumBackingType, VertexAttribute) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all parameters should be named in a function
typedef NS_ENUM(EnumBackingType, VertexAttribute) | |
typedef NS_ENUM(EnumBackingType /*unused*/, VertexAttribute /*unused*/) |
VertexAttributeTexcoord = 1, | ||
}; | ||
|
||
typedef NS_ENUM(EnumBackingType, TextureIndex) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all parameters should be named in a function
typedef NS_ENUM(EnumBackingType, TextureIndex) | |
typedef NS_ENUM(EnumBackingType /*unused*/, TextureIndex /*unused*/) |
TextureIndexColor = 0, | ||
}; | ||
|
||
typedef struct |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use using
instead of typedef
typedef struct | |
using Uniforms = struct |
{ | ||
matrix_float4x4 projectionMatrix; | ||
matrix_float4x4 modelViewMatrix; | ||
} Uniforms; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use using
instead of typedef
} Uniforms; | |
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clang-Tidy
found issue(s) with the introduced code (2/2)
TextureIndexColor = 0, | ||
}; | ||
|
||
typedef struct |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
constructor does not initialize these fields: projectionMatrix, modelViewMatrix
@@ -26,6 +26,7 @@ | |||
#include <fstream> | |||
#include <memory> | |||
#include <utility> | |||
#include <CoreFoundation/CoreFoundation.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CoreFoundation/CoreFoundation.h
file not found
Hi, @dbratcher. I know you said you were busy and wouldn't have much time to look into. If you ever do, could you please address the build check comments? Thank you! |
Sets up an iOS project and experiments with getting resources to load in the iOS system