On 21/03/2016 17:02, rutu.shah...@gmail.com wrote:
>
> Hi,
> This patch replaces get_ticks_per_sec() calls to NANOSECONDS_PER_SECOND.
> Also, as there are no callers, get_ticks_per_sec() has been removed.
> Replacement imporves readability and understandability of code.
>
> Example given by P
From: Rutuja Shah
Hi,
This patch replaces get_ticks_per_sec() calls to NANOSECONDS_PER_SECOND. Also,
as there are no callers, get_ticks_per_sec() has been removed. Replacement
imporves readability and understandability of code.
Example given by Paolo Bonzini,
timer_mod(fdctrl->result_timer,
On 03/20/2016 07:31 PM, rutu.shah...@gmail.com wrote:
> From: Rutuja Shah
Can you add the patch description (with examples of why this is the right thing
to do) here and not in the cover letter?
>
> Signed-off-by: Rutuja Shah
>
> ---
> audio/audio.c | 3 +--
> audio/noaudio.c
On Mon, Mar 21, 2016 at 12:01 AM, wrote:
Hi,
Some lines seem to have been skipped some how. This patch replaces
get_ticks_per_sec() calls to NANOSECONDS_PER_SECOND. Also, as there
are no callers, get_ticks_per_sec() has been removed. Replacement
imporves readability and understandability of code.
From: Rutuja Shah
Signed-off-by: Rutuja Shah
---
audio/audio.c | 3 +--
audio/noaudio.c | 8
audio/spiceaudio.c| 4 ++--
audio/wavaudio.c | 2 +-
backends/baum.c | 2 +-
block/qed.c | 2 +-
cpus.c
Example given by Paolo Bonzini,
timer_mod(fdctrl->result_timer,
qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) + (get_ticks_per_sec() / 50));
NANOSECONDS_PER_SECOND makes it obvious that the timer will expire in 1/50th of
a second.
---
audio/audio.c | 3 +--
audio/noaudio.c
