From fee217551c12754db517d1badd448fc9064a48a8 Mon Sep 17 00:00:00 2001 From: Nathan Scott Date: Thu, 26 Nov 2020 16:15:09 +1100 Subject: Drop unneeded parameters to the ScreenManager constructor All calls to ScreenManager_new always pass the same first five values, the orientation is always HORIZONTAL and the y1 parameter is always the height of the passed-in header struct pointer. I think its safe to assert at this point that no VERTICAL orientation will arrive (if it does, its no harm in re-adding this then) - so we can remove unused conditionals (and TODOs) based on orientation too. --- htop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'htop.c') diff --git a/htop.c b/htop.c index 15b58304..2c22b14f 100644 --- a/htop.c +++ b/htop.c @@ -326,7 +326,7 @@ int main(int argc, char** argv) { if (flags.commFilter) setCommFilter(&state, &(flags.commFilter)); - ScreenManager* scr = ScreenManager_new(0, header->height, 0, -1, HORIZONTAL, header, settings, &state, true); + ScreenManager* scr = ScreenManager_new(header, settings, &state, true); ScreenManager_add(scr, (Panel*) panel, -1); ProcessList_scan(pl, false); -- cgit v1.2.3