Skip to content
Snippets Groups Projects

feat(epicardium): read card10.cfg

Merged swym requested to merge swym/firmware:config into master

Adds simple config parser along with config_ API that:

  • supports default values for options
  • allows typed querying of config values
  • types supported: boolean, integer, floating point and string

unknown options are ignored and LOG_WARNed on the console

So it seems I suck at writing parsers, the main loop is a little wonky but it works.

Tested:

  • int, bool, double option types
  • option.names.with.dots
  • whitespace all over the place
  • empty card10.cfg
Edited by swym

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 6 - Changes Requested label and removed 6 - Needs Review label

  • rahix changed milestone to %v1.9

    changed milestone to %v1.9

  • swym added 134 commits

    added 134 commits

    • 0a5dc41a...37da10f9 - 131 commits from branch card10:master
    • 54cb7169 - feat(epicardium): read card10.cfg
    • 57bfade4 - lifecycle: use execulte_elf option from card10.cfg
    • 4a0631b9 - card10.cfg: seek back in the file instead of memmoving around

    Compare with previous version

  • swym added 1 commit

    added 1 commit

    • 58d60e81 - card10.cfg: load on epicardium startup

    Compare with previous version

  • Looks good to me! Thanks for keeping our codebase clean.

    However, I'm only a bot - so a human will still have to approve this request.

  • swym added 1 commit

    added 1 commit

    • fe154963 - card10.cfg: have strtol detect base, fix zero-termination bug for

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading