[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#64516: [PATCH] docview: Only enable imenu when supported
From: |
Eli Zaretskii |
Subject: |
bug#64516: [PATCH] docview: Only enable imenu when supported |
Date: |
Sat, 15 Jul 2023 11:05:11 +0300 |
> From: Morgan Smith <Morgan.J.Smith@outlook.com>
> Cc: Eli Zaretskii <eliz@gnu.org>, 64516@debbugs.gnu.org, jao@gnu.org
> Date: Tue, 11 Jul 2023 14:22:49 -0400
>
> I apologize for my "oops I figured it out immediately after I sent the
> email" syndrome. I was trying to find the answer in the bug reports
> when the answer was in the code and quite obvious.
>
> Anyways here is a patch that unconditionally enables imenu but checks
> the return value of mutool and will raise an imenu-unavailable-error
> when it is not successful. However, that error stops docview from
> displaying the document so I decided to put the setup function in a hook
> so the error occurs after the document is displayed.
>
> I looked briefly for documentation on how errors work in hooks but could
> not find any. I don't want this error to prevent the other hooks from
> being run (which would be user defined hooks. we don't currently use
> this hook).
Tassilo, is this okay, in your opinion?
- bug#64516: [PATCH] docview: Only enable imenu when supported, Morgan Smith, 2023/07/07
- bug#64516: [PATCH] docview: Only enable imenu when supported, Morgan Smith, 2023/07/07
- bug#64516: [PATCH] docview: Only enable imenu when supported, Eli Zaretskii, 2023/07/08
- bug#64516: [PATCH] docview: Only enable imenu when supported, Tassilo Horn, 2023/07/09
- bug#64516: [PATCH] docview: Only enable imenu when supported, Morgan Smith, 2023/07/11
- bug#64516: [PATCH] docview: Only enable imenu when supported, Morgan Smith, 2023/07/11
- bug#64516: [PATCH] docview: Only enable imenu when supported,
Eli Zaretskii <=
- bug#64516: [PATCH] docview: Only enable imenu when supported, Tassilo Horn, 2023/07/15
- bug#64516: [PATCH] docview: Only enable imenu when supported, Morgan Smith, 2023/07/15
- bug#64516: [PATCH] docview: Only enable imenu when supported, Tassilo Horn, 2023/07/16
- bug#64516: [PATCH] docview: Only enable imenu when supported, Morgan Smith, 2023/07/16
- bug#64516: [PATCH] docview: Only enable imenu when supported, Tassilo Horn, 2023/07/16
- bug#64516: [PATCH] docview: Only enable imenu when supported, Morgan Smith, 2023/07/18
- bug#64516: [PATCH] docview: Only enable imenu when supported, Tassilo Horn, 2023/07/18
- bug#64516: [PATCH] docview: Only enable imenu when supported, Eli Zaretskii, 2023/07/20
- bug#64516: [PATCH] docview: Only enable imenu when supported, Basil L. Contovounesios, 2023/07/28
- bug#64516: [PATCH] docview: Only enable imenu when supported, Tassilo Horn, 2023/07/28