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

Remove PROCESS_RECEIVE_NO_TRUDP callback that was never called. #58

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Si1ver
Copy link
Collaborator

@Si1ver Si1ver commented Mar 16, 2020

This change removes PROCESS_RECEIVE_NO_TRUDP that was never called for non-Trudp packets but was called for Trudp packets with unknown packet type and valid checksum.

@lgtm-com
Copy link

lgtm-com bot commented Mar 16, 2020

This pull request fixes 1 alert when merging 36638bb8090f4fbbeb44a7e24c33165d1360329d into 1b381f8 - view on LGTM.com

fixed alerts:

  • 1 for FIXME comment

src/trudp.c Outdated Show resolved Hide resolved
src/trudp.c Outdated Show resolved Hide resolved
@kirill-scherba kirill-scherba added the invalid This doesn't seem right label Mar 23, 2020
@Si1ver Si1ver changed the title Fix PROCESS_RECEIVE_NO_TRUDP callback was never called. WIP Fix PROCESS_RECEIVE_NO_TRUDP callback was never called. Mar 24, 2020
@Si1ver Si1ver force-pushed the fix/process-revceive-notrudp-callback branch from e39528c to 49af505 Compare April 19, 2020 07:10
@Si1ver Si1ver changed the title WIP Fix PROCESS_RECEIVE_NO_TRUDP callback was never called. Remove PROCESS_RECEIVE_NO_TRUDP callback that was never called. Apr 19, 2020
@Si1ver Si1ver force-pushed the fix/process-revceive-notrudp-callback branch from 49af505 to 7096933 Compare April 19, 2020 07:14
@lgtm-com
Copy link

lgtm-com bot commented Apr 19, 2020

This pull request fixes 1 alert when merging 70969339e8ff4a6f3e8af6d0f6c04618343887d1 into bb21ebf - view on LGTM.com

fixed alerts:

  • 1 for FIXME comment

@Si1ver Si1ver removed the invalid This doesn't seem right label Apr 19, 2020
@Si1ver Si1ver force-pushed the fix/process-revceive-notrudp-callback branch from 7096933 to 9a9ca03 Compare April 19, 2020 07:39
@lgtm-com
Copy link

lgtm-com bot commented Apr 19, 2020

This pull request fixes 1 alert when merging 9a9ca03 into bb21ebf - view on LGTM.com

fixed alerts:

  • 1 for FIXME comment

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.

3 participants