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

Extract ES2K NEIGHBOR_MOD_TABLE definitions (3/5) #135

Merged
merged 1 commit into from
May 29, 2024
Merged

Conversation

ffoulkes
Copy link
Contributor

  • Extracted the LNWv3 NEIGHBOR_MOD_TABLE definitions into a separate header file.

- Extracted the LNWv3 NEIGHBOR_MOD_TABLE definitions into a
  separate header file.

Signed-off-by: Derek Foster <[email protected]>
Copy link
Collaborator

@5abeel 5abeel left a comment

Choose a reason for hiding this comment

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

LGTM

@5abeel
Copy link
Collaborator

5abeel commented May 29, 2024

  • Extracted the LNWv3 NEIGHBOR_MOD_TABLE definitions into a separate header file.

One comment here - looks like the PR is only addressing LNWv2 - typo in the main comment?

@ffoulkes ffoulkes merged commit 79e9b5b into main May 29, 2024
4 checks passed
@ffoulkes ffoulkes deleted the neighbor_mod_table branch May 29, 2024 14:42
@ffoulkes
Copy link
Contributor Author

One comment here - looks like the PR is only addressing LNWv2 - typo in the main comment?

You're right. the NEIGHBOR_MOD_TABLE is specific to LNWv2. There's a typo in the 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.

2 participants