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

Removing unused variables #223

Open
wants to merge 3 commits into
base: 5.8.7.1_35809.20191129_COEX20191120-7777
Choose a base branch
from
Open

Removing unused variables #223

wants to merge 3 commits into from

Commits on Oct 2, 2022

  1. Purge unused variables 

    Sweep across default build to purge variables marked as unused.
This is incomplete and there are more items which need guarding off,
for example there are variables which are only used by debug print
macros that do nothing by default.
    @sevan
    sevan committed Oct 2, 2022
    Configuration menu
    Copy the full SHA
    1658e83 View commit details
    Browse the repository at this point in the history

Commits on Oct 3, 2022

  1. Add CONFIG_RTW_DEBUG guards 

    There are variables which are only used by debug print macros when
debugging is enabled. Guard them off with CONFIG_RTW_DEBUG to keep
the compiler happy about unused variables.
    @sevan
    sevan committed Oct 3, 2022
    Configuration menu
    Copy the full SHA
    7ea31bb View commit details
    Browse the repository at this point in the history

Commits on Oct 4, 2022

  1. Add CONFIG_RTW_ACS/CONFIG_RTW_ACS_DBG guards 

    while here zap some unused variable declarations.
    @sevan
    sevan committed Oct 4, 2022
    Configuration menu
    Copy the full SHA
    5fe4fff View commit details
    Browse the repository at this point in the history