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

Move CharBuffer class into its own file so it can be shared by #300

Merged

Conversation

jamessynge
Copy link
Contributor

power_board and telemetry_board sketches.

power_board and telemetry_board sketches.
Copy link
Member

@wtgee wtgee left a comment

Choose a reason for hiding this comment

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

👍

uint8_t len = 0;
while (!Empty() && isdigit(Peek())) {
char c = Next();
v = v * 10 + c - '0';
Copy link
Member

Choose a reason for hiding this comment

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

What's actually going on here? I can't seem to parse it.

@codecov
Copy link

codecov bot commented Jan 5, 2018

Codecov Report

Merging #300 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #300   +/-   ##
========================================
  Coverage    83.84%   83.84%           
========================================
  Files           51       51           
  Lines         3367     3367           
  Branches       422      422           
========================================
  Hits          2823     2823           
  Misses         404      404           
  Partials       140      140

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9628289...84a30f5. Read the comment docs.

@jamessynge jamessynge merged commit 18c469b into panoptes:develop Jan 5, 2018
@jamessynge jamessynge deleted the issue-289-share-char-buffer branch January 7, 2018 20:26
@wtgee wtgee mentioned this pull request Jan 24, 2018
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