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

Finishing off the rest of src/*.rs #455

Merged
merged 19 commits into from
Jun 23, 2017
Merged

Conversation

Susurrus
Copy link
Contributor

After this it's only the source files in lib/ that need to be ported.

98eab59 needs careful review since the datatype should actually be a UNION2!, but I'm not certain how to do that.

8a1d180 I also need some help with since I didn't figure out how to use UNION2! to port sockaddr_in6.

Note that this builds off of #448 and #454, which is why it looks so huge. This is actually only the commits since 2233a23.

@Susurrus
Copy link
Contributor Author

There are some problems with porting over some of the interfaces where they don't have a UUID in the header describing them. Specifically: vswriter::IVssWriterComponents, vswriter::IVssWMFiledesc, vswriter::IVssWMDependency, vswriter::IVssWMDependency, and vsbackup::IVssWMComponent.

Is there something I'm missing or should these be declared differently?

@Susurrus
Copy link
Contributor Author

This should now build as I'm using all-zeroes for missing UUIDs and I cleaned up some other issues.

Also, I redid in6_addr as a UNION2. I think it's correct, but I'd need confirmation on that.

@Susurrus Susurrus force-pushed the even_moar branch 2 times, most recently from 4f634b1 to 2c9e63e Compare June 9, 2017 16:00
@retep998 retep998 merged commit 1c195c6 into retep998:dev Jun 23, 2017
@Susurrus Susurrus deleted the even_moar branch June 23, 2017 20:04
@Susurrus
Copy link
Contributor Author

@retep998 Thanks!

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