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

Fix date format to show correct day of month #754

Merged
merged 1 commit into from
Jan 7, 2020

Conversation

bessman
Copy link
Contributor

@bessman bessman commented Jan 7, 2020

ASCWriter currently writes the month twice (once as three-letter abbreviation and once as zero-padded decimal number), but does not write the day of month. I don't think this is correct. For example, today's date (2020-01-07) is currently written Tue Jan 01 01:14:48.625694 PM 2020.

With this fix, today's date will instead be written Tue Jan 07 01:14:48.625694 PM 2020.

%m is month as a zero-padded decimal number
%d is day of the month as a zero-padded decimal number
@codecov
Copy link

codecov bot commented Jan 7, 2020

Codecov Report

Merging #754 into develop will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           develop     #754   +/-   ##
========================================
  Coverage    69.52%   69.52%           
========================================
  Files           70       70           
  Lines         6485     6485           
========================================
  Hits          4509     4509           
  Misses        1976     1976

@bessman
Copy link
Contributor Author

bessman commented Jan 7, 2020

For reference, CANalyzer writes today's date in .asc-files thus:
Tue Jan 07 01:14:48.626 pm 2020

Copy link
Collaborator

@christiansandberg christiansandberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That seems reasonable. Don’t know why I got it wrong in the first place and why nobody noticed it before...

@hardbyte hardbyte merged commit 148921a into hardbyte:develop Jan 7, 2020
@christiansandberg christiansandberg mentioned this pull request Feb 29, 2020
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.

None yet

3 participants