From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Adam Plaice Newsgroups: gmane.emacs.bugs Subject: bug#29584: imagemagick-types-inhibit does not result in correct image backend (minor) Date: Wed, 6 Dec 2017 19:32:12 +0000 Message-ID: References: <588tefbtn1.fsf@fencepost.gnu.org> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Trace: blaine.gmane.org 1512591047 22292 195.159.176.226 (6 Dec 2017 20:10:47 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Wed, 6 Dec 2017 20:10:47 +0000 (UTC) Cc: 29584@debbugs.gnu.org To: Glenn Morris Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Wed Dec 06 21:10:43 2017 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1eMg1f-0005bl-QA for geb-bug-gnu-emacs@m.gmane.org; Wed, 06 Dec 2017 21:10:39 +0100 Original-Received: from localhost ([::1]:57539 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eMg1m-00058C-MM for geb-bug-gnu-emacs@m.gmane.org; Wed, 06 Dec 2017 15:10:46 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:59272) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eMfRL-0000VD-QF for bug-gnu-emacs@gnu.org; Wed, 06 Dec 2017 14:33:08 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eMfRG-00007P-TG for bug-gnu-emacs@gnu.org; Wed, 06 Dec 2017 14:33:07 -0500 Original-Received: from debbugs.gnu.org ([208.118.235.43]:40788) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1eMfRG-00006z-PO for bug-gnu-emacs@gnu.org; Wed, 06 Dec 2017 14:33:02 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1eMfRG-0002jF-Ah for bug-gnu-emacs@gnu.org; Wed, 06 Dec 2017 14:33:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Adam Plaice Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 06 Dec 2017 19:33:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 29584 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 29584-submit@debbugs.gnu.org id=B29584.151258874210418 (code B ref 29584); Wed, 06 Dec 2017 19:33:02 +0000 Original-Received: (at 29584) by debbugs.gnu.org; 6 Dec 2017 19:32:22 +0000 Original-Received: from localhost ([127.0.0.1]:49469 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1eMfQa-0002fL-H8 for submit@debbugs.gnu.org; Wed, 06 Dec 2017 14:32:22 -0500 Original-Received: from mail-yb0-f173.google.com ([209.85.213.173]:36762) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1eMfQY-0002aM-Mx for 29584@debbugs.gnu.org; Wed, 06 Dec 2017 14:32:19 -0500 Original-Received: by mail-yb0-f173.google.com with SMTP id j7so2020045ybl.3 for <29584@debbugs.gnu.org>; Wed, 06 Dec 2017 11:32:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=lRbxRgRwK48b4puZnteLFLmZVlX+KBjSdVzZSd9xhrs=; b=gvbqvO33sgWYgZrZi0g5DRf1aGfBywQyouY1wZaz7L/wWbwsAHnQiRBouePuWxMSSK gU2Xfiz/eL4HomjH67iRshX/YANzUtZCBdAPIP7G1hsggwQFq43Hmv11Opc9ocoOY5v8 ovz+X8AmbiN7zij1P3Qh4RtlGP/abbhJHpJwN7a7wUSMPADRTixWjdBruTFhV4POHH8k 5hONpOlKsn6Y/R5G9sD/2rS0ljlGnEY7zhGtx4jEcKUI5FYV9Mnf0NHtDEj4RKhljgNE p3xd4NH6CqhA1gC7SyQjChYQ/zOVdOz+mYG23ybyubcmXdLrZgS0FLUcaRPqTxGmx05U CBrg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=lRbxRgRwK48b4puZnteLFLmZVlX+KBjSdVzZSd9xhrs=; b=jtV/9B3hhhamj5DGstrB7oewNBAa7QcsUsojO2BeXrpViYbDZ/vp1NSERhj6lS78gN wJtyfWTEVgc/KN1nYEmvcybBuyctz/XOfLrI8YFEPY2a4Dn7uKSNoBTNH2l0xE8GGfgV 4vQbeBCAW6laBm5bi/FZ/Cem02wKxQO09zj1aWie197rMgvRWX3D+hpfodid+8Gk1KMu R0T+sTmyttWx2BjR95lYZQK9Lh6xEOBs5CZVw4MiXP2piadfMMCYQjWuJc6KYA1Egk2S kmbujgbNz9pVIN7XqDAFXX0Ab9oKFv0f1j0yoCQTWeMmHwp+UW2hugcjszbt5qif80Uj YobQ== X-Gm-Message-State: AJaThX5V061uLfcAT3y2eG3unvwhqZCtCgUGbGweMajVw/YwBAR+GjSr P58VNtF8U6yZbXXZrQSPM9U6Qnrb2/yHPfVHKWU= X-Google-Smtp-Source: AGs4zMZr1RHbxjqpRj3q+py3vS+atYZs9c0eUyxxpR+YpNaf2QX4BfOEj6crI1XQynhhrI0JgEHNi/y6k/LTRectaao= X-Received: by 10.37.172.154 with SMTP id x26mr15729518ybi.22.1512588733151; Wed, 06 Dec 2017 11:32:13 -0800 (PST) Original-Received: by 10.37.129.201 with HTTP; Wed, 6 Dec 2017 11:32:12 -0800 (PST) In-Reply-To: <588tefbtn1.fsf@fencepost.gnu.org> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:140763 Archived-At: Thanks for the clarification. As it's a deliberate change and to avoid over-complication that nobody usually needs (if one generally dislikes the imagemagick backend then one should use a version of emacs compiled without it, and in the very rare case when one does need to change the backend on the fly, one can just override image-toggle-display-image), leaving the code alone is probably the best option. However, I'd argue that the documentation should be updated =E2=80=94 the docstring of imagemagick-types-inhibit already contains extensive warnings " If you change this without using customize, you must call =E2=80=98imagemagick-register-types=E2=80=99 afterwards. If Emacs is compiled without ImageMagick support, this variable has no effect. " so another sentence to the effect of "If emacs is compiled with alternative image backends, in addition to ImageMagick support, this variable also has no effect." wouldn't be out place (and since there are already extensive warnings, creating an impression of completeness, the absence of an important warning =E2=80=94 having multiple= sets of backends is probably the most frequent situation on GNU/Linux =E2=80=94 would be confusing).