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 IAR warning: Warning[Pe381]: extra ";" ignored #1836

Merged
merged 2 commits into from
Jan 6, 2023

Conversation

HiFiPhile
Copy link
Collaborator

Describe the PR
Seems IAR is not happy with empty ; in struct

typedef struct {
  bool is_host; // host or device most
  union {
      uint8_t daddr;
      uint8_t rhport;
      uint8_t hwid;
  };
  uint8_t ep_addr;
  uint8_t ep_speed;

  uint16_t ep_packetsize;
  uint16_t ep_bufsize;

  // TODO xfer_fifo can skip this buffer
  uint8_t* ep_buf;

  tu_fifo_t ff;

  // mutex: read if ep rx, write if e tx
  OSAL_MUTEX_DEF(ff_mutex);

}tu_edpt_stream_t;

Additional context
https://stackoverflow.com/questions/18611303/warning-extra-ignored-by-using-c-macros-with-armcc

@HiFiPhile
Copy link
Collaborator Author

Hummm it's either GCC or IAR not happy :(

Copy link
Owner

@hathach hathach left a comment

Choose a reason for hiding this comment

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

look great, thank you

@hathach hathach merged commit cde16b8 into hathach:master Jan 6, 2023
@HiFiPhile HiFiPhile deleted the Pe381 branch March 31, 2023 06:51
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