-
-
Notifications
You must be signed in to change notification settings - Fork 389
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
Add full set of struct tests + changes to make them pass #607
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.
Thanks for putting all the work into this PR. Just watch out for those spaces.
src/um/commdlg.rs
Outdated
nFromPage: DWORD, | ||
nToPage: DWORD, | ||
}} | ||
pub type LPPRINTPAGERANGE = *mut PRINTPAGERANGE; | ||
pub type PCPRINTPAGERANGE = *const PRINTPAGERANGE; | ||
STRUCT!{struct PRINTDLGEXA { | ||
STRUCT!{#[cfg_attr(target_arch = "x86", repr(packed))] struct PRINTDLGEXA { |
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.
Are these... double spaces I see? You monster!
src/um/setupapi.rs
Outdated
@@ -32,25 +32,25 @@ pub const MAX_SERVICE_NAME_LEN: usize = 256; | |||
pub const MAX_SUBTITLE_LEN: usize = 256; | |||
pub const SP_MAX_MACHINENAME_LENGTH: usize = MAX_PATH + 3; | |||
pub type HINF = PVOID; | |||
STRUCT!{struct INFCONTEXT { | |||
STRUCT!{#[cfg_attr(target_arch = "x86", repr(packed))] struct INFCONTEXT { |
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.
And even more of those double spaces!
Ever since winapi version 0.3.5, tty detection for msys terminals has been broken. The definition for FILE_NAME_INFO struct changed in retep998/winapi-rs#607, causing the first character from the filename to be cut off.
Ever since winapi version 0.3.5, tty detection for msys terminals has been broken. The definition for FILE_NAME_INFO struct changed in retep998/winapi-rs#607, causing the first character from the filename to be cut off.
* Add full set of struct tests + changes to make them pass * Update tests + fixes * Update struct tests * Update tests + fixes * Update tests + changes * Fix double spaces
Issue #442.
Fix summary:
One unresolved issue is SPPHRASE in sapi. The C and C++ definitions differ from each other and I'm unsure as to which is the correct definition. The C++ definition uses base classes which add padding the C definition does not.
Would you like the C code used to generate the tests added in as well?