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

Order ports by lowest PID to highest #8

Merged
merged 4 commits into from
Feb 2, 2025

Conversation

samuelballantyne
Copy link
Contributor

Ordering ports from lowest to highest PID creates fewer ports and allows for port ordering on all systems.
Duplicates do not stop assignment; they only trigger a warning. Each device, even if it shares the same PID as another, still receives a separate port index.

Ordering ports from lowest to highest PID creates fewer ports and allows for port ordering on all systems.
Copy link
Owner

@SeongGino SeongGino left a comment

Choose a reason for hiding this comment

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

Am still on holiday for a few more days, so I can't test to be sure or even look very closely from mobile. Looks good so far aside from this thing I noticed - will re-review when I return either Thursday or Friday.

Copy link
Owner

@SeongGino SeongGino left a comment

Choose a reason for hiding this comment

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

Everything else looks good, except for this one thing here.

Copy link
Owner

@SeongGino SeongGino left a comment

Choose a reason for hiding this comment

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

Looks good to me now!

@SeongGino SeongGino merged commit 7fad621 into SeongGino:main Feb 2, 2025
2 checks passed
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