From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Tassilo Horn Newsgroups: gmane.emacs.devel Subject: Re: Using the ImageMagick backend seems to leak memory Date: Tue, 11 Jan 2011 20:55:29 +0100 Message-ID: <87y66r6wr2.fsf@member.fsf.org> References: <87sjx6zczl.fsf@member.fsf.org> <30611405.post@talk.nabble.com> <874o9l86eb.fsf@member.fsf.org> <87zkr7nxo8.fsf@member.fsf.org> <8739oz8dei.fsf@member.fsf.org> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: dough.gmane.org 1294775843 19627 80.91.229.12 (11 Jan 2011 19:57:23 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Tue, 11 Jan 2011 19:57:23 +0000 (UTC) Cc: Emacs-devel@gnu.org, Jashy , Miles Bader To: joakim@verona.se Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Jan 11 20:57:17 2011 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1PckLC-0004cg-LO for ged-emacs-devel@m.gmane.org; Tue, 11 Jan 2011 20:57:15 +0100 Original-Received: from localhost ([127.0.0.1]:52391 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PckLC-0002Sc-07 for ged-emacs-devel@m.gmane.org; Tue, 11 Jan 2011 14:57:14 -0500 Original-Received: from [140.186.70.92] (port=39488 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PckKO-0002M4-3e for Emacs-devel@gnu.org; Tue, 11 Jan 2011 14:57:10 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PckJY-0007JU-Vl for Emacs-devel@gnu.org; Tue, 11 Jan 2011 14:56:23 -0500 Original-Received: from out1.smtp.messagingengine.com ([66.111.4.25]:60229) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PckJY-0007JG-NP; Tue, 11 Jan 2011 14:55:32 -0500 Original-Received: from compute3.internal (compute3.nyi.mail.srv.osa [10.202.2.43]) by gateway1.messagingengine.com (Postfix) with ESMTP id D501320507; Tue, 11 Jan 2011 14:55:31 -0500 (EST) Original-Received: from frontend1.messagingengine.com ([10.202.2.160]) by compute3.internal (MEProxy); Tue, 11 Jan 2011 14:55:31 -0500 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=messagingengine.com; h=from:to:cc:subject:references:date:in-reply-to:message-id:mime-version:content-type; s=smtpout; bh=ChzWeW+XzhqkfzxXqG4vhfvKOeI=; b=YMkJ6Co/0IMuo0+xcxsB+XsLKlOFdpPKozS6TAjuQE2/SbDSUj0KkIlYmPcoHnGI6dJLS8t873fO3W8YKQBI8fx99Z+WWDExw3uL8nsjcL3v95idT0sUShDddTMRm0+Z72EVOe2c98k84AwR3xg1Nle05f/80hgddayOJVRVNNk= X-Sasl-enc: rd2IG3CJza1MY8U6SrDzNkDL1DvLYrq0yHdDKCqjd56v 1294775731 Original-Received: from thinkpad (95-88-32-105-dynip.superkabel.de [95.88.32.105]) by mail.messagingengine.com (Postfix) with ESMTPA id 8A990403BF3; Tue, 11 Jan 2011 14:55:30 -0500 (EST) In-Reply-To: (joakim@verona.se's message of "Tue, 11 Jan 2011 20:35:52 +0100") User-Agent: Gnus/5.110011 (No Gnus v0.11) Emacs/24.0.50 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:134454 Archived-At: joakim@verona.se writes: Hi! >> Anyway, should I commit the following patch which includes your >> DestroyImage() call that fixes the issue, and I additionally >> surrounded image loading with calls to >> MagickWandGenesis()/MagickWandTerminus(), because that's the way the >> API is used throughout the example codes in the MagickWand API docs? > > The patch seems fine. Ok, I'll commit it in some minutes. > Im not entirely sure about genesis/terminus though, I interpreted the > docs to mean genesis=library load. The docs say initialize and terminate the MagickWand environment whatever that means. At least the docs indicate that the calls come in pairs, and till now the terminus function was never called. Bye, Tassilo