@bunnybot merge

Diff comments:

> 
> === modified file 'src/ui_basic/panel.cc'
> --- src/ui_basic/panel.cc     2016-01-27 08:06:40 +0000
> +++ src/ui_basic/panel.cc     2016-01-30 22:45:22 +0000
> @@ -150,15 +147,9 @@
>  
>       // Panel-specific startup code. This might call end_modal()!
>       start();
> -     g_gr->update();
>  
> -     uint32_t minTime;
> -     {
> -             int maxfps = g_options.pull_section("global").get_int("maxfps", 
> 25);
> -             if (maxfps < 5)
> -                     maxfps = 5;
> -             minTime = 1000 / maxfps;
> -     }
> +     const uint32_t minimum_frame_time =
> +        1000 / std::max(5, 
> g_options.pull_section("global").get_int("maxfps", 30));

25 makes no real sense. Screens today all work on 60 Hz virtual refresh rate. 
We can target half of that, but everything else, but 25 is very arbitrary. I 
want to reach 60 eventually, but for now, 30 seems okayish.

>  
>       while (_running) {
>               const uint32_t startTime = SDL_GetTicks();


-- 
https://code.launchpad.net/~widelands-dev/widelands/remove_update/+merge/284524
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/remove_update.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to