Skip to content
This repository was 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
OndrejBakan wants to merge 1 commit into
hexchat:masterfrom
OndrejBakan:patch-1
Open

Allow chanlist_minusers to be 0 to show empty channels#2064
OndrejBakan wants to merge 1 commit into
hexchat:masterfrom
OndrejBakan:patch-1

Conversation

@OndrejBakan
Copy link
Copy Markdown

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
Comment thread src/fe-gtk/chanlist.c
/* 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
Copy Markdown
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
Copy Markdown
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.

Comment thread src/fe-gtk/chanlist.c
@@ -727,7 +727,7 @@ chanlist_opengui (server *serv, int do_refresh)

if (!serv->gui->chanlist_minusers)
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

2 participants