From: Eli Zaretskii <eliz@gnu.org>
To: "Harald Jörg" <haj@posteo.de>
Cc: larsi@gnus.org, stefankangas@gmail.com, 17948@debbugs.gnu.org
Subject: bug#17948: 24.3; cperl: Needs an equivalent to `c-file-style' [PATCH]
Date: Mon, 08 Jan 2024 14:51:16 +0200 [thread overview]
Message-ID: <837ckkq98b.fsf@gnu.org> (raw)
In-Reply-To: <87frz8jks8.fsf_-_@oook.m.uunet.de> (message from Harald Jörg on Mon, 08 Jan 2024 08:23:35 +0000)
> Cc: Lars Ingebrigtsen <larsi@gnus.org>, Stefan Kangas <stefankangas@gmail.com>
> From: Harald Jörg <haj@posteo.de>
> Date: Mon, 08 Jan 2024 08:23:35 +0000
>
> So... I promised a bit too much. It is not possible to override an
> individual indentation setting (e.g. cperl-indent-level) when a
> cperl-file-style is used. File (and directory) variables are applied
> after the body of cperl-mode, and the styles do their work only after
> that in hack-local-variables-hook. I did not know that. So, styles
> will always override the settings they provide.
>
> But apart from that, the patch (attached) does what I would expect: I
> can set a default style in custom, individual styles from .dir-locals.el
> or file local variables work without affecting other buffers.
>
> I suggest this patch should go to the emacs-29 branch.
Sorry, no new commands/features on the release branch at this time.
This should go to master.
Thanks.
next prev parent reply other threads:[~2024-01-08 12:51 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-05 20:10 bug#17948: 24.3; cperl: Needs an equivalent to `c-file-style' Samuel Bronson
2022-05-11 14:29 ` Lars Ingebrigtsen
2024-01-02 17:33 ` Stefan Kangas
2024-01-03 19:02 ` Harald Jörg
2024-01-05 20:18 ` Stefan Kangas
2024-01-08 8:23 ` bug#17948: 24.3; cperl: Needs an equivalent to `c-file-style' [PATCH] Harald Jörg
2024-01-08 12:51 ` Eli Zaretskii [this message]
2024-01-08 15:26 ` Harald Jörg
2024-01-08 19:27 ` Stefan Kangas
2024-01-09 16:06 ` bug#17948: 24.3; cperl: Needs an equivalent to `c-file-style' john muhl via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-01-09 18:23 ` Harald Jörg
2024-03-08 8:13 ` bug#17948: Emacs bug control: close 17948 Harald Jörg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=837ckkq98b.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=17948@debbugs.gnu.org \
--cc=haj@posteo.de \
--cc=larsi@gnus.org \
--cc=stefankangas@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).