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.

Sign up for GitHub

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

Jump to bottom

Implement RegKeyMetadata::last_write_time for RegKey::query_info #25

Merged
merged 1 commit into from
Jul 17, 2019
Merged

Implement RegKeyMetadata::last_write_time for RegKey::query_info #25

merged 1 commit into from
Jul 17, 2019

Conversation

poiru
Copy link
Contributor

@poiru poiru commented Jul 14, 2019

This enables the Debug and Default traits for the winapi crate because
they are needed to use FILETIME.

@poiru
Implement RegKeyMetadata::last_write_time for RegKey::query_info
c63318b 
This enables the Debug and Default traits for the winapi crate because
they are needed to use `FILETIME`.
@poiru
Copy link
Contributor Author

poiru commented Jul 14, 2019 •
edited
Loading

I just noticed #22. This is a simpler version, but #22 may be preferable if we don't want to expose FILETIME as-is. Personally I would prefer this PR because people can easily implement a filetime_to_date function.

cc @gentoo90

@gentoo90 gentoo90 merged commit c63318b into gentoo90:master Jul 17, 2019
@poiru poiru deleted the query-info-last-write-time branch July 17, 2019 18:14
@gentoo90
Copy link
Owner

Published to crates.io

Changelog

0.6.1

  • Add last_write_time field to RegKeyMetadata (returned by RegKey::query_info()) (#25).
  • Add get_last_write_time_system() and get_last_write_time_chrono() (under chrono feature) methods to RegKeyMetadata.

@poiru
Copy link
Contributor Author

poiru commented Jul 17, 2019

Sweet, thank you! I guess #22 can be closed?

@gentoo90 gentoo90 mentioned this pull request Jul 17, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@poiru @gentoo90