Skip to content
This repository has been archived by the owner on Feb 10, 2024. It is now read-only.

Allow chanlist_minusers to be 0 to show empty channels #2064

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

Conversation

OndrejBakan
Copy link

Hello,

I think it should be allowed to show channels with 0 users in /list. I tried to make this change and it is working for me, but I think it could be in the official release too.

Allow chanlist_minusers to be 0 to show empty channels
@@ -310,8 +310,8 @@ chanlist_do_refresh (server *serv)
else
{
/* download all, filter minusers locally only */
serv->p_list_channels (serv, "", 1);
serv->gui->chanlist_minusers_downloaded = 1;
serv->p_list_channels (serv, "", 0);
Copy link
Member

Choose a reason for hiding this comment

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

The idea seems fine but I'm not sure about this line. I never want us to actually request that many potential channels unless the user explicitly does so.

Copy link
Author

Choose a reason for hiding this comment

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

That's fine. My IRC network is some kind of bridged network, so it works for me, but I understand it probably won't work for real IRC networks with tons of channels.

@@ -727,7 +727,7 @@ chanlist_opengui (server *serv, int do_refresh)

if (!serv->gui->chanlist_minusers)
Copy link
Member

@TingPing TingPing Oct 2, 2017

Choose a reason for hiding this comment

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

0 is a valid value for this now.

There might be more places this assumption is made too.

@OndrejBakan
Copy link
Author

@TingPing Sorry to bother you, but after compiling (I am really noob at it, I just edited the files and then ran build in VS) Ctrl+W shortcut does not work. It worked until that change. Any idea why?

@TingPing
Copy link
Member

TingPing commented Oct 4, 2017

Ctrl+W shortcut does not work. Any idea why?

That shortcut was disabled months ago, you can add a custom one in Settings > Keyboard Shortcuts that calls /close.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

None yet

2 participants