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 typo #8280

Closed
wants to merge 1 commit into from
Closed

fix typo #8280

wants to merge 1 commit into from

Conversation

takkanm
Copy link

@takkanm takkanm commented Mar 1, 2020

Description

I can't build on current master. 1ec8a72

Compiling: quantum/quantum.c                                                                       quantum/quantum.c: In function 'send_dword':
quantum/quantum.c:646:5: error: 'uint16_tword' undeclared (first use in this function); did you mean 'uint16_t'?
     uint16_tword = (number >> 16);
     ^~~~~~~~~~~~
     uint16_t

So, I think uint16_tword is typo.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

No Issues

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@gwjo
Copy link

gwjo commented Mar 1, 2020

Dup of #8282

@fauxpark
Copy link
Member

fauxpark commented Mar 1, 2020

Thanks for your contribution! This has been superseded by 8282, which fixes the issue "properly" - the bot would have just changed it back after merging.

It seems to be a bug in clang-format 7 that occurs when you have a variable declaration as the first statement of a function with a comment at the end of the line. Very bizarre.

@fauxpark fauxpark closed this Mar 1, 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.

3 participants