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

Fixes issue with value extraction from raw message #10

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Fixes issue with value extraction from raw message #10

wants to merge 11 commits into from

Commits on Jun 4, 2021

  1. An additional check since the length of the PgnList[i].FieldList look… 

    …s to change throughout the loop. To avoid runtime errors then this seems to work.
    @asbjorn
    asbjorn committed Jun 4, 2021
    Configuration menu
    Copy the full SHA
    739f92e View commit details
    Browse the repository at this point in the history

  2. Merge pull request #1 from asbjorn/parsepacket-runtime-error 

    An additional check since the length of the PgnList[i].FieldList look…
    @asbjorn
    asbjorn authored Jun 4, 2021
    Configuration menu
    Copy the full SHA
    1b8bf3f View commit details
    Browse the repository at this point in the history

Commits on Aug 19, 2021

  1. New bugfix to ensure that we don't go beyond the length of PgnList

    @asbjorn
    asbjorn committed Aug 19, 2021
    Configuration menu
    Copy the full SHA
    f50ce07 View commit details
    Browse the repository at this point in the history

Commits on Sep 14, 2021

  1. Merge pull request #2 from timmathews/master 

    Update from upstream
    @asbjorn
    asbjorn authored Sep 14, 2021
    Configuration menu
    Copy the full SHA
    17d2f5e View commit details
    Browse the repository at this point in the history

  2. Changed to asbjorn for internal binary to work

    @asbjorn
    asbjorn committed Sep 14, 2021
    Configuration menu
    Copy the full SHA
    79d3751 View commit details
    Browse the repository at this point in the history

Commits on Feb 18, 2022

  1. Ran go mod tidy

    @asbjorn
    asbjorn committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    c9694a0 View commit details
    Browse the repository at this point in the history

  2. New testcase which triggers when incorrect extractNumber for NMEA dat… 

    …a occurs.
    @asbjorn
    asbjorn committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    902f4fc View commit details
    Browse the repository at this point in the history

  3. Included a commented code block to check and validate the extracted n… 

    …umber.

If value is too large (overflowed) the function
`func (msg *RawMessage) extractNumber(..)` returns an error and the
value will be set to nil.
    @asbjorn
    asbjorn committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    c4cc278 View commit details
    Browse the repository at this point in the history

  4. Updated version and some dependencies

    @asbjorn
    asbjorn committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    a85dccd View commit details
    Browse the repository at this point in the history

  5. Updated go modules dependencies

    @asbjorn
    asbjorn committed Feb 18, 2022
    Configuration menu
    Copy the full SHA
    a985633 View commit details
    Browse the repository at this point in the history

Commits on Feb 21, 2022

  1. Fixes timmathews#9 by introducing max value checking as done in githu… 

    …b.com/canboat/canboat
    @asbjorn
    asbjorn committed Feb 21, 2022
    Configuration menu
    Copy the full SHA
    2bce74b View commit details
    Browse the repository at this point in the history