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

Update projdefs.h (IDFGH-6179) #7856

Closed
wants to merge 1 commit into from
Closed

Update projdefs.h (IDFGH-6179) #7856

wants to merge 1 commit into from

Conversation

lisekt84
Copy link
Contributor

@lisekt84 lisekt84 commented Nov 8, 2021

Lose of precision on pdTICKS_TO_MS convertions. (IDFGH-6176) #7853

This bug is on master and other branches too

Lose of precision on pdTICKS_TO_MS convertions. (IDFGH-6176) #7853
@espressif-bot espressif-bot added the Status: Opened Issue is new label Nov 8, 2021
@github-actions github-actions bot changed the title Update projdefs.h Update projdefs.h (IDFGH-6179) Nov 8, 2021
@KaeLL
Copy link
Contributor

KaeLL commented Nov 8, 2021

PRs like this are supposed to be made against the master branch.

Copy link
Contributor

@Dazza0 Dazza0 left a comment

Choose a reason for hiding this comment

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

  • Nitpick: Commit message format
freertos: Add higher precision pdTICKS_TO_MS macro

Closes https://github.com/espressif/esp-idf/issues/7853
Closes https://github.com/espressif/esp-idf/pull/7856

@lisekt84 Left a couple of nitpicks but the new macro LGTM.

@Dazza0 Dazza0 assigned Dazza0 and unassigned zikalino Nov 23, 2021
@espressif-bot espressif-bot added Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new Resolution: NA Issue resolution is unavailable labels Nov 29, 2021
espressif-bot pushed a commit that referenced this pull request Dec 1, 2021
Freertos: Fix loss of precision in pdTICKS_TO_MS (GitHub PR #7856)

Closes IDFGH-6179

See merge request espressif/esp-idf!16133
espressif-bot pushed a commit that referenced this pull request Jan 8, 2022
Closes #7856
Closes #7853

[darian@espressif.com: Updated commit message]
Signed-off-by: Darian Leung <darian@espressif.com>
espressif-bot pushed a commit that referenced this pull request Jan 26, 2022
Closes #7856
Closes #7853

[darian@espressif.com: Updated commit message]
Signed-off-by: Darian Leung <darian@espressif.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants