swaybar: allow null status_command

[RyanDwyer]

Sway sets a default status_command which runs date every second. This patch removes this behaviour so the user can have a NULL status bar if desired.

I had to swap swaybar's event_loop_poll and wl_display_flush so that it would map the initial surface.

[ddevault]

Can you add the old default status command to the default config?

[ianyfan]

Maybe you should add a check in status_line_init for a NULL cmd?

[RyanDwyer]

status_line_init is not called if the status command is NULL.

sway/swaybar/bar.c

Lines 499 to 501 in 89f4ebe

	if (bar->config->status_command) {
	bar->status = status_line_init(bar->config->status_command);
	}

[emersion]

There should be a way to change status_command to be NULL in the config. Maybe status_command -?

[RyanDwyer]

You can remove or comment the directive.

[emersion]

I mean, there should be a way to change it via swaymsg. This is related to #2790

[RyanDwyer]

Done. Note that I found and fixed a bug in cmd_bar which prevented the anytime commands from being run at runtime.

[ddevault]

Thanks!

[ianyfan]

I found and fixed a bug in cmd_bar which prevented the anytime commands from being run at runtime.

I think this was actually right before. It's just that the array names bar_handlers and bar_config_handlers seem to be the wrong way round for what they denote.