Re: [Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-22 Thread Miklos Vajna
On Thu, Jan 22, 2015 at 10:58:13AM +0100, Tobias Madl tobias.madl@gmail.com wrote: But i don't get the thing with changing it to a C++11 enum class. It is already an enum class In your feature branch, you have: enum class IdlePriority { That's a C++11 enum class. On master, it's:

[Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-22 Thread Tobias Madl
include/vcl/idle.hxx| 41 - vcl/source/app/idle.cxx |4 ++-- 2 files changed, 10 insertions(+), 35 deletions(-) New commits: commit d2382e1a8f04ac91768fe59e1498048f3fcfdc9a Author: Tobias Madl tobias.madl@gmail.com Date: Thu Jan 22 12:27:45

Re: [Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-22 Thread Tobias Madl
Removed the Prefix :) 2dc1136e2287b52ba3c3c58a1d70de78ad53fef7 Cheers Tobi On Thu, Jan 22, 2015 at 12:14 PM, Miklos Vajna vmik...@collabora.co.uk wrote: On Thu, Jan 22, 2015 at 10:58:13AM +0100, Tobias Madl tobias.madl@gmail.com wrote: But i don't get the thing with changing it to a

Re: [Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-22 Thread Tobias Madl
So i replaced it with a static cast now :) But i don't get the thing with changing it to a C++11 enum class. It is already an enum class Cheers Tobi On Thu, Jan 22, 2015 at 10:41 AM, Miklos Vajna vmik...@collabora.co.uk wrote: Hi Tobias, [ Taking back the discussion to the list, so

Re: [Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-22 Thread Miklos Vajna
Hi Tobias, On Wed, Jan 21, 2015 at 05:06:25AM -0800, Tobias Madl tobias.madl@gmail.com wrote: enum class IdlePriority { -VCL_IDLE_PRIORITY_STARVATIONPROTECTION = -1, // Do not use this for normal prioritizing! - -VCL_IDLE_PRIORITY_HIGHEST = 0, // - 0ms -

Re: [Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-22 Thread Miklos Vajna
Hi Tobias, [ Taking back the discussion to the list, so Noel and others can jump in. ] On Thu, Jan 22, 2015 at 10:20:59AM +0100, Tobias Madl tobias.madl@gmail.com wrote: Replacing sounds good, but i have to change it in every single file, that uses them, so there is the question if this

[Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-21 Thread Tobias Madl
include/vcl/idle.hxx| 51 vcl/source/app/idle.cxx | 68 2 files changed, 53 insertions(+), 66 deletions(-) New commits: commit 75d61d16fce70a53d250a0ef6b7df30b1fd329e6 Author: Tobias Madl

[Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-20 Thread Jennifer Liebel
include/vcl/idle.hxx| 23 +-- vcl/source/app/idle.cxx | 29 + 2 files changed, 30 insertions(+), 22 deletions(-) New commits: commit bc637c2096621061d1a5dc37aaebc657f195ac7f Author: Jennifer Liebel jliebe...@gmail.com Date: Tue Jan 20

[Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-19 Thread Tobias Madl
include/vcl/idle.hxx| 13 ++--- vcl/source/app/idle.cxx | 11 --- 2 files changed, 14 insertions(+), 10 deletions(-) New commits: commit 7d40d5fbf0464725f85b24c1dcbe568b3a162c35 Author: Tobias Madl tobias.madl@gmail.com Date: Mon Jan 19 16:30:23 2015 +

[Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2015-01-16 Thread Tobias Madl
include/vcl/idle.hxx | 22 +++- include/vcl/timer.hxx|1 vcl/source/app/idle.cxx | 51 +-- vcl/source/app/timer.cxx | 25 +++ 4 files changed, 88 insertions(+), 11 deletions(-) New commits: commit

[Libreoffice-commits] core.git: Branch 'feature/priorities' - include/vcl vcl/source

2014-10-30 Thread Jennifer Liebel
include/vcl/timer.hxx|2 +- vcl/source/app/timer.cxx | 32 ++-- 2 files changed, 23 insertions(+), 11 deletions(-) New commits: commit b5d780f20aaa9f70bf53adde32bcb10ec57e171c Author: Jennifer Liebel jliebe...@gmail.com Date: Thu Oct 30 13:46:44 2014 +