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

SD_ProtocolData.pm: fixes #106 #570

Merged
merged 4 commits into from
Apr 23, 2019
Merged

SD_ProtocolData.pm: fixes #106 #570

merged 4 commits into from
Apr 23, 2019

Conversation

sidey79
Copy link
Contributor

@sidey79 sidey79 commented Apr 22, 2019

removed zero key from protocol 76 to avoid perl warning fixes: #156

  • Please check if the PR fulfills these requirements
  • Tests for the changes have been added / modified (needed for for bug fixes / features)
  • commandref has been added / updated (needed for bug fixes / features)
  • CHANGED has been updated (needed for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

remove key zero which has no value under protocol 76

  • What is the current behavior? (You can also link to an open issue here)

key zero has an empty list

  • What is the new behavior (if this is a feature change)?

key does not exists anymore

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

no

removed zero key from protocol 76 to avoid perl warning fixes: #156
@sidey79 sidey79 merged commit 8c7b1cb into dev-r34 Apr 23, 2019
@sidey79 sidey79 deleted the fiix_156 branch April 23, 2019 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants