[0/5] qemu-timer: Clean code

Am 23.04.2012 22:13, schrieb Anthony Liguori:
> On 04/20/2012 05:03 AM, Stefan Weil wrote:
>> These patches remove timer code which is no longer needed
>> and try to improve the remaining code.
>>
>> [PATCH 1/5] qemu-timer: Remove redundant include statements
>> [PATCH 2/5] qemu-timer: Remove unused function qemu_alarm_pending
>> [PATCH 3/5] qemu-timer: Use bool, false, true for boolean values
>> [PATCH 4/5] qemu-timer: Remove function alarm_has_dynticks
>> [PATCH 5/5] qemu-timer: Optimize data structures
>>
>>
>
> If you're goign to rev a patch, please resubmit the full series.  
> Otherwise it's a pain in the ass to apply because the patches have to 
> be reordered by hand. It's not so bad if it's the last patch in a 
> series but when it's in the middle, it sucks.
>
> Regards,
>
> Anthony Liguori

The patches 4 and 5 don't depend on each other, so they can be reordered.
Then patch 4 (v2) would be the last patch.

To avoid further pain, it's now possible to pull this series.

Regards,

Stefan Weil


The following changes since commit 092dfc7714ad7983aeb0cada5d5983e9fde8d84c:

   Merge remote-tracking branch 'origin/master' into staging (2012-04-23 
15:02:53 -0500)

are available in the git repository at:


   git://qemu.weilnetz.de/qemu.git timer

for you to fetch changes up to 475d53efedeaf3444e78088939afd376e8843d09:

   qemu-timer: Optimize data structures (2012-04-23 22:37:12 +0200)

----------------------------------------------------------------
Stefan Weil (5):
       qemu-timer: Remove redundant include statements
       qemu-timer: Remove unused function qemu_alarm_pending
       qemu-timer: Use bool, false, true for boolean values
       qemu-timer: Remove function alarm_has_dynticks
       qemu-timer: Optimize data structures

  qemu-timer.c |   94 
++++++++++++++++++---------------------------------------
  qemu-timer.h |   13 ++------
  2 files changed, 33 insertions(+), 74 deletions(-)

On Mon, Apr 23, 2012 at 20:39, Stefan Weil <sw@weilnetz.de> wrote:
> Am 23.04.2012 22:13, schrieb Anthony Liguori:
>>
>> On 04/20/2012 05:03 AM, Stefan Weil wrote:
>>>
>>> These patches remove timer code which is no longer needed
>>> and try to improve the remaining code.
>>>
>>> [PATCH 1/5] qemu-timer: Remove redundant include statements
>>> [PATCH 2/5] qemu-timer: Remove unused function qemu_alarm_pending
>>> [PATCH 3/5] qemu-timer: Use bool, false, true for boolean values
>>> [PATCH 4/5] qemu-timer: Remove function alarm_has_dynticks
>>> [PATCH 5/5] qemu-timer: Optimize data structures
>>>
>>>
>>
>> If you're goign to rev a patch, please resubmit the full series.
>>  Otherwise it's a pain in the ass to apply because the patches have to be
>> reordered by hand. It's not so bad if it's the last patch in a series but
>> when it's in the middle, it sucks.
>>
>> Regards,
>>
>> Anthony Liguori
>
>
> The patches 4 and 5 don't depend on each other, so they can be reordered.
> Then patch 4 (v2) would be the last patch.
>
> To avoid further pain, it's now possible to pull this series.

Thanks, pulled.

>
> Regards,
>
> Stefan Weil
>
>
> The following changes since commit 092dfc7714ad7983aeb0cada5d5983e9fde8d84c:
>
>  Merge remote-tracking branch 'origin/master' into staging (2012-04-23
> 15:02:53 -0500)
>
> are available in the git repository at:
>
>
>  git://qemu.weilnetz.de/qemu.git timer
>
> for you to fetch changes up to 475d53efedeaf3444e78088939afd376e8843d09:
>
>  qemu-timer: Optimize data structures (2012-04-23 22:37:12 +0200)
>
> ----------------------------------------------------------------
> Stefan Weil (5):
>      qemu-timer: Remove redundant include statements
>      qemu-timer: Remove unused function qemu_alarm_pending
>      qemu-timer: Use bool, false, true for boolean values
>      qemu-timer: Remove function alarm_has_dynticks
>      qemu-timer: Optimize data structures
>
>  qemu-timer.c |   94
> ++++++++++++++++++---------------------------------------
>  qemu-timer.h |   13 ++------
>  2 files changed, 33 insertions(+), 74 deletions(-)
>
>