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.

Sign up for GitHub

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

Jump to bottom

Fix unused variables in RTEMS #260

Closed
skliper opened this issue Sep 30, 2019 · 7 comments
Closed

Fix unused variables in RTEMS #260

skliper opened this issue Sep 30, 2019 · 7 comments
Labels
bug
Milestone
Aquila

Comments

@skliper
Copy link
Contributor

skliper commented Sep 30, 2019

Compiling with -Wall -Werror noted two unused variables in OSAL for rtems-ng. These are simple fixes to clean up.
@skliper skliper added this to the osal-5.0.0 milestone Sep 30, 2019
@skliper skliper self-assigned this Sep 30, 2019
@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Imported from trac issue 237. Created by jphickey on 2019-08-20T13:29:12, last modified: 2019-08-29T15:38:05

@skliper skliper added the bug label Sep 30, 2019
@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2019-08-20 13:36:24:

Fix is in commit [changeset:8f583484] for review

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2019-08-23 16:31:51:

I would like to see this included in the 5.0 release so it builds warning-free. Please reset to a later milestone if not in agreement.

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jhageman on 2019-08-26 09:57:02:

Approved for fast-track, will code review in CCB prior to merge.

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2019-08-27 16:49:06:

Merged to branch ic-20190826-fasttrack for integration test. Scheduled for review at next CCB.

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jhageman on 2019-08-29 13:46:12:

CCB 8/28/2019 - Code reviewed and approved for merge to development branch

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2019-08-29 15:38:05:

Merged to development branch

@skliper skliper closed this as completed Sep 30, 2019
@skliper skliper modified the milestones: osal-5.0.0, 5.0.0 Sep 30, 2019
@skliper skliper removed their assignment Sep 30, 2019
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@skliper
Fix nasa#260, Remove unused network_includes.h
d3e1cec
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@astrogeco
Merge pull request nasa#1067 from skliper/fix260-rm_network_includes
fb18780 
Fix nasa#260, Remove unused network_includes.h
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
Aquila
Development

No branches or pull requests
1 participant
@skliper