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

Work on new DateTime native side #336

Merged
merged 2 commits into from May 28, 2017
Merged

Work on new DateTime native side #336

merged 2 commits into from May 28, 2017

Conversation

ghost
Copy link

@ghost ghost commented May 27, 2017

Removal of TimeZone (see issue #335)
Rework on DateTime structure to remove method related to UTC.local conversions, since nanoFramework is only using UTC.
Moved some managed methods to the native side, using ChibiOS API where possible.

This PR is meant to be used with the changes made in nf-class-libraries regarding DateTime as well. The PR in nf-class-libraries is the managed counterpart of this one.

Signed-off-by: Christophe Gerbier [email protected]

Removal of TimeZone (see issue #335)
Rework on DateTime structure to remove method related to UTC.local conversions, since nanoFramework is only using UTC.
Moved some managed methods to the native side, using ChibiOS API where possible.

This PR is meant to be used with the changes made in nf-class-libraries regarding DateTime as well. The PR in nf-class-libraries is the managed counterpart of this one.

Signed-off-by: Christophe Gerbier <[email protected]>
@nfbot
Copy link
Member

nfbot commented May 27, 2017

Hi @MikroBusNet,

I'm nanoFramework bot.
Thank you for your contribution!

A human will be reviewing it shortly. 😉

On boards other than MBN QUAIL.
Use of RTC has now to be TRUE in halconf.h

Signed-off-by: Christophe Gerbier <[email protected]>
@josesimoes josesimoes merged commit 06903fd into nanoframework:master May 28, 2017
@ghost ghost deleted the NativeDateTime branch May 28, 2017 18:50
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