Skip to content

Align the SPEED column - #10

Open
BillyDonahue wants to merge 5 commits into
JPEWdev:masterfrom
BillyDonahue:speed_column_align
Open

Align the SPEED column#10
BillyDonahue wants to merge 5 commits into
JPEWdev:masterfrom
BillyDonahue:speed_column_align

Conversation

@BillyDonahue

Copy link
Copy Markdown

The primary change is to get the SPEED column to line its decimal points up.
Part of doing that was breaking down the SIMPLE_COLUMN macro into pure C++.

I think your for loops need reference indexes so I fixed those too.
Otherwise you copy each element into a loop variable unnecessarily.

@coveralls

coveralls commented Jun 10, 2020

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 95

  • 28 of 62 (45.16%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 42.591%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/main.cpp 1 3 33.33%
src/draw.cpp 27 59 45.76%
Totals Coverage Status
Change from base Build 93: 0.4%
Covered Lines: 411
Relevant Lines: 965

💛 - Coveralls

@BillyDonahue BillyDonahue changed the title Speed column align Align the SPEED column Jun 10, 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.

2 participants