From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=34950 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OMmI6-0007Sc-90 for qemu-devel@nongnu.org; Thu, 10 Jun 2010 14:15:51 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OMmI2-0008Gk-QZ for qemu-devel@nongnu.org; Thu, 10 Jun 2010 14:15:46 -0400 Received: from mail-px0-f173.google.com ([209.85.212.173]:41689) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OMmI2-0008GU-Kp for qemu-devel@nongnu.org; Thu, 10 Jun 2010 14:15:42 -0400 Received: by pxi2 with SMTP id 2so85951pxi.4 for ; Thu, 10 Jun 2010 11:15:41 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20100527053847.GD31807@valinux.co.jp> References: <20100527053847.GD31807@valinux.co.jp> From: Blue Swirl Date: Thu, 10 Jun 2010 18:15:21 +0000 Message-ID: Subject: Re: [Qemu-devel] [PATCH] main: allocate gui_timer only once. Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Isaku Yamahata Cc: qemu-devel@nongnu.org Thanks, applied. On Thu, May 27, 2010 at 5:38 AM, Isaku Yamahata wr= ote: > fix memory leak. > there is no need to allocate more than one gui_timer. > > Signed-off-by: Isaku Yamahata > --- > =C2=A0vl.c | =C2=A0 =C2=A01 + > =C2=A01 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/vl.c b/vl.c > index 417554f..22cdf43 100644 > --- a/vl.c > +++ b/vl.c > @@ -3794,6 +3794,7 @@ int main(int argc, char **argv, char **envp) > =C2=A0 =C2=A0 =C2=A0 =C2=A0 if (dcl->dpy_refresh !=3D NULL) { > =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ds->gui_timer =3D qemu_new_time= r(rt_clock, gui_update, ds); > =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 qemu_mod_timer(ds->gui_timer, q= emu_get_clock(rt_clock)); > + =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0break; > =C2=A0 =C2=A0 =C2=A0 =C2=A0 } > =C2=A0 =C2=A0 =C2=A0 =C2=A0 dcl =3D dcl->next; > =C2=A0 =C2=A0 } > -- > 1.6.6.1 > > >