Skip to content

Native cleanup#227

Merged
mehlis merged 1 commit intoRIOT-OS:masterfrom
LudwigKnuepfer:native_cleanup
Oct 2, 2013
Merged

Native cleanup#227
mehlis merged 1 commit intoRIOT-OS:masterfrom
LudwigKnuepfer:native_cleanup

Conversation

@LudwigKnuepfer
Copy link
Copy Markdown
Member

make some variables volatile
remove rx_buffer_next from header
more DEBUG output
fix warnings
coding conventions
change puts to DEBUG
rename end context/stack

make some variables volatile
remove rx_buffer_next from header
more DEBUG output
fix warnings
coding conventions
change puts to DEBUG
rename end-context/stack
@OlegHahm
Copy link
Copy Markdown
Member

OlegHahm commented Oct 1, 2013

ACK

@mehlis
Copy link
Copy Markdown
Contributor

mehlis commented Oct 2, 2013

ACK and go

mehlis added a commit that referenced this pull request Oct 2, 2013
@mehlis mehlis merged commit f34d10e into RIOT-OS:master Oct 2, 2013
bors bot added a commit that referenced this pull request May 11, 2023
19577: cpu/esp32: fix remaining compilation errors for GCC 12.2 r=maribu a=gschorcht

### Contribution description

This PR fixes one problem of CI compilation.

With [PR #227](RIOT-OS/riotdocker#227) `riotdocker` was updated to GCC 12.2 . Obviously, following parts were not covered by compilation test before:

- [ ] `esp_eth`
- [ ] `esp_hw_counter`
- [ ] `esp_can`

### Testing procedure

### Issues/PRs references


Co-authored-by: Gunar Schorcht <[email protected]>
bors bot added a commit that referenced this pull request May 11, 2023
19577: cpu/esp32: fix remaining compilation errors for GCC 12.2 r=maribu a=gschorcht

### Contribution description

This PR fixes one problem of CI compilation.

With [PR #227](RIOT-OS/riotdocker#227) `riotdocker` was updated to GCC 12.2 . Obviously, following parts were not covered by compilation test before:

- [ ] `esp_eth`
- [ ] `esp_hw_counter`
- [ ] `esp_can`

### Testing procedure

### Issues/PRs references


Co-authored-by: Gunar Schorcht <[email protected]>
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