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

Overhaul datetime() scene language function. #436

Open
wants to merge 4 commits into
base: release/v3.8.0
Choose a base branch
from

Conversation

c-lipka
Copy link
Contributor

@c-lipka c-lipka commented Aug 13, 2021

  • Added local_time BOOL global setting, defaulting to off; when set to on, datetime() will now report local time instead of UTC.
  • datetime() will now sanity-check the specified format string (unless the default is used), and generate various warnings if applicable.
  • datetime() will now try to distinguish between a genuine error, a and non-error situation in which the result string just happens to be empty.
  • LC_TIME locale is now set someplace more sane.

Also, pushed back on the use of boost date_time library in favor of C++11 features.

- Add `local_time BOOL` global setting, defaulting to `off`; when set to `on`, `datetime` will report local time instead of UTC.
- `datetime` will now sanity-check the specified format string (unless the default is used), and generate various warnings if applicable.
- `datetime` will now try to distinguish between a genuine error, a and non-error situation in which the result string just happens to be empty.
- `LC_TIME` locale is now set someplace more sane.

Also, pushed back on the use of boost date_time library in favor of C++11 features.
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.

1 participant