Skip to content

Does not show some of the interfaces #2

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
Open
websahibi opened this issue Dec 28, 2020 · 2 comments
Open

Does not show some of the interfaces #2

websahibi opened this issue Dec 28, 2020 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@websahibi
Copy link

I very much like to use this utility but I am sad about the interface limitations. I think it is limited to 7 because if there is more interfaces, they do not show up
@eribertomota eribertomota added enhancement New feature or request help wanted Extra attention is needed labels Jan 31, 2021
@jwbowen
Copy link
Contributor

jwbowen commented Oct 18, 2023

I'm running into this as well. If I have the time in the next couple of days I might take a shot at a patch.

jwbowen added a commit to jwbowen/cbm that referenced this issue Oct 30, 2023
@jwbowen
Increase max rows to display
e68ae27 
Intended to address resurrecting-open-source-projects#2
about only showing 7 interfaces.

The max number of rows was just a "magic number" set to 8 with a //TODO
next to it. I've changed it to a #define and set it to 64. Admittedly
this is also arbitrary, but it's less likely to cut off useful data
than 8.
@jwbowen jwbowen mentioned this issue Oct 30, 2023
Merged
@jwbowen
Copy link
Contributor

jwbowen commented Oct 30, 2023

Admittedly I know very little about proper defensive coding practices with (n)curses applications, but for my limited use of wanting a quick way to see the tx/rx per interface on more than 7 interfaces, this seems to do the trick.

@websahibi, I know it's been almost three years since you opened this, but FYI :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jwbowen @eribertomota @websahibi