From: john muhl via "Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
To: "Harald Jörg" <haj@posteo.de>
Cc: Eli Zaretskii <eliz@gnu.org>,
larsi@gnus.org, 17948@debbugs.gnu.org, stefankangas@gmail.com
Subject: bug#17948: 24.3; cperl: Needs an equivalent to `c-file-style'
Date: Tue, 09 Jan 2024 10:06:33 -0600 [thread overview]
Message-ID: <87bk9uwkxi.fsf_-_@pub.pink> (raw)
In-Reply-To: <871qarkfs7.fsf@oook.m.uunet.de> ("Harald Jörg"'s message of "Mon, 08 Jan 2024 15:26:16 +0000")
[-- Attachment #1: Type: text/plain, Size: 673 bytes --]
Harald Jörg <haj@posteo.de> writes:
> Eli Zaretskii writes:
>
>>> 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
>>> [...]
>>> 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.
>
> Thanks for reviewing!
>
> I understand - and have now pushed the patch to master.
This commit causes a couple of cperl-mode indent tests to start
failing. Attached is the log from running:
make -k -C test check-lisp-progmodes
[-- Attachment #2: cperl-mode-tests.log --]
[-- Type: text/plain, Size: 8160 bytes --]
Running 57 tests (2024-01-09 10:00:55-0600, selector `(not (or (tag :expensive-test) (tag :unstable)))')
passed 1/57 cperl-test-attribute-list-rx (0.000238 sec)
passed 2/57 cperl-test-attribute-rx (0.000127 sec)
passed 3/57 cperl-test-bug-11733 (0.002425 sec)
passed 4/57 cperl-test-bug-11996 (0.001372 sec)
passed 5/57 cperl-test-bug-14343 (0.002349 sec)
passed 6/57 cperl-test-bug-16368 (0.000173 sec)
passed 7/57 cperl-test-bug-19709 (0.001242 sec)
passed 8/57 cperl-test-bug-22355 (0.000636 sec)
passed 9/57 cperl-test-bug-23992 (0.001337 sec)
passed 10/57 cperl-test-bug-25098 (0.000780 sec)
passed 11/57 cperl-test-bug-28650 (0.001259 sec)
passed 12/57 cperl-test-bug-30393 (0.001014 sec)
passed 13/57 cperl-test-bug-35925 (0.001159 sec)
End of `/ ... /' string/RE not found: (scan-error Unbalanced parentheses 7 13)
passed 14/57 cperl-test-bug-37127 (0.977492 sec)
passed 15/57 cperl-test-bug-42168 (0.001213 sec)
passed 16/57 cperl-test-bug-45255 (0.000506 sec)
passed 17/57 cperl-test-bug-47112 (0.001242 sec)
passed 18/57 cperl-test-bug-47598 (0.000947 sec)
passed 19/57 cperl-test-bug-64190 (0.005074 sec)
passed 20/57 cperl-test-bug-64364 (0.005063 sec)
passed 21/57 cperl-test-bug-65834 (0.000903 sec)
passed 22/57 cperl-test-bug-66145 (0.004820 sec)
passed 23/57 cperl-test-bug-66161 (0.001057 sec)
passed 24/57 cperl-test-bug-997 (0.000413 sec)
passed 25/57 cperl-test-fontify-attrs-and-signatures (0.003491 sec)
passed 26/57 cperl-test-fontify-class (0.000867 sec)
passed 27/57 cperl-test-fontify-declarations (0.000347 sec)
passed 28/57 cperl-test-fontify-punct-vars (0.001012 sec)
passed 29/57 cperl-test-fontify-special-variables (0.000342 sec)
passed 30/57 cperl-test-fontify-sub-names (0.002409 sec)
Scanning for "hard" Perl constructions...
End of here-document `HERE' not found.
End of here-document `HERE' not found.
Scanning for "hard" Perl constructions...
End of here-document `THERE' not found.
End of here-document `THERE' not found.
Scanning for "hard" Perl constructions... done
passed 31/57 cperl-test-here-doc-missing-end (0.001749 sec)
passed 32/57 cperl-test-heredocs (0.041433 sec)
Precede char by C-q to avoid expansion
passed 33/57 cperl-test-hyperactive-electric-else (0.002132 sec)
passed 34/57 cperl-test-identifier-rx (0.000499 sec)
passed 35/57 cperl-test-identify-heredoc (0.001102 sec)
passed 36/57 cperl-test-identify-no-heredoc (0.001232 sec)
passed 37/57 cperl-test-imenu-index (0.003937 sec)
Test cperl-test-indent-exp backtrace:
signal(ert-test-failed (((should (equal got expected)) :form (equal
ert-fail(((should (equal got expected)) :form (equal "test case whil
(if (unwind-protect (setq value-7 (apply fn-5 args-6)) (setq form-de
(let (form-description-9) (if (unwind-protect (setq value-7 (apply f
(let ((value-7 'ert-form-evaluation-aborted-8)) (let (form-descripti
(let* ((fn-5 #'equal) (args-6 (condition-case err (list got expected
(progn (insert code) (goto-char (point-min)) (funcall cperl-test-mod
(unwind-protect (progn (insert code) (goto-char (point-min)) (funcal
(save-current-buffer (set-buffer temp-buffer) (unwind-protect (progn
(let ((temp-buffer (generate-new-buffer " *temp*" t))) (save-current
(let ((name (match-string 1)) (code (match-string 2)) (expected (mat
(while (re-search-forward (concat "^# ?-+ \\_<\\(?1:.+?\\)\\_>: inpu
(progn (insert-file-contents (expand-file-name "cperl-indent-exp.pl"
(unwind-protect (progn (insert-file-contents (expand-file-name "cper
(save-current-buffer (set-buffer temp-buffer) (unwind-protect (progn
(let ((temp-buffer (generate-new-buffer " *temp*" t))) (save-current
(closure (t) nil (let* ((fn-0 #'eq) (args-1 (condition-case err (lis
#f(compiled-function () #<bytecode 0x1fe51f30ad42b54d>)()
handler-bind-1(#f(compiled-function () #<bytecode 0x1fe51f30ad42b54d
ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
ert-run-test(#s(ert-test :name cperl-test-indent-exp :documentation
ert-run-or-rerun-test(#s(ert--stats :selector ... :tests ... :test-m
ert-run-tests((not (or (tag :expensive-test) (tag :unstable))) #f(co
ert-run-tests-batch((not (or (tag :expensive-test) (tag :unstable)))
ert-run-tests-batch-and-exit((not (or (tag :expensive-test) (tag :un
eval((ert-run-tests-batch-and-exit '(not (or (tag :expensive-test) (
command-line-1(("-L" ":." "-l" "ert" "-l" "lisp/progmodes/cperl-mode
command-line()
normal-top-level()
Test cperl-test-indent-exp condition:
(ert-test-failed
((should (equal got expected)) :form
(equal
"test case while loop:\n{\n while (1) {\n\11say \"boring loop\";\n }\n continue {\n\11last;\n }\n}\n"
"test case while loop:\n{\n while (1) {\n say \"boring loop\";\n } continue {\n last;\n }\n}\n")
:value nil :explanation
(arrays-of-different-length 96 92
"test case while loop:\n{\n while (1) {\n\11say \"boring loop\";\n }\n continue {\n\11last;\n }\n}\n"
"test case while loop:\n{\n while (1) {\n say \"boring loop\";\n } continue {\n last;\n }\n}\n"
first-mismatch-at 26)))
FAILED 38/57 cperl-test-indent-exp (0.002078 sec) at lisp/progmodes/cperl-mode-tests.el:100
passed 39/57 cperl-test-indent-styles (0.006402 sec)
passed 40/57 cperl-test-package-regexp (0.000558 sec)
passed 41/57 cperl-test-prototype-rx (0.000626 sec)
passed 42/57 cperl-test-signature-rx (0.000521 sec)
passed 43/57 cperl-test-unicode-arrays (0.003492 sec)
passed 44/57 cperl-test-unicode-fhs (0.001231 sec)
passed 45/57 cperl-test-unicode-hashes (0.003823 sec)
passed 46/57 cperl-test-unicode-hashkeys (0.002064 sec)
passed 47/57 cperl-test-unicode-hashref (0.000865 sec)
passed 48/57 cperl-test-unicode-labels (0.000802 sec)
passed 49/57 cperl-test-unicode-proto (0.000885 sec)
passed 50/57 cperl-test-unicode-sub (0.000936 sec)
passed 51/57 cperl-test-unicode-varname (0.000588 sec)
passed 52/57 cperl-test-unicode-varname-list (0.001282 sec)
passed 53/57 cperl-test-version-regexp (0.000345 sec)
passed 54/57 cperl-test-word-at-point (0.000193 sec)
passed 55/57 cperl-test-ws+-rx (0.000268 sec)
passed 56/57 cperl-test-ws-rx (0.000167 sec)
Test test-indentation backtrace:
signal(ert-test-failed (("Mismatch in test \"cperl-indent1\", file /
ert-fail(("Mismatch in test \"cperl-indent1\", file /home/jm/src/ema
ert-test--erts-test(((dummy . t) (code)) "/home/jm/src/emacs/test/li
ert-test-erts-file("/home/jm/src/emacs/test/lisp/progmodes/cperl-mod
(closure (perl-indent-parens-as-block perl-indent-level perl-continu
#f(compiled-function () #<bytecode 0x1fe51f30ad42b54d>)()
handler-bind-1(#f(compiled-function () #<bytecode 0x1fe51f30ad42b54d
ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
ert-run-test(#s(ert-test :name test-indentation :documentation nil :
ert-run-or-rerun-test(#s(ert--stats :selector ... :tests ... :test-m
ert-run-tests((not (or (tag :expensive-test) (tag :unstable))) #f(co
ert-run-tests-batch((not (or (tag :expensive-test) (tag :unstable)))
ert-run-tests-batch-and-exit((not (or (tag :expensive-test) (tag :un
eval((ert-run-tests-batch-and-exit '(not (or (tag :expensive-test) (
command-line-1(("-L" ":." "-l" "ert" "-l" "lisp/progmodes/cperl-mode
command-line()
normal-top-level()
Test test-indentation condition:
(ert-test-failed
("Mismatch in test \"cperl-indent1\", file /home/jm/src/emacs/test/lisp/progmodes/cperl-mode-resources/cperl-indents.erts"
"{\n print \"\",\n\11\"\",\n\11foo::bar(),\n\11\"\";\n}\n"
"{\n print \"\",\n \"\",\n foo::bar(),\n \"\";\n}\n"))
FAILED 57/57 test-indentation (0.001704 sec) at lisp/progmodes/cperl-mode-tests.el:1436
Ran 57 tests, 55 results as expected, 2 unexpected (2024-01-09 10:00:57-0600, 1.338509 sec)
2 unexpected results:
FAILED cperl-test-indent-exp
FAILED test-indentation
next prev parent reply other threads:[~2024-01-09 16:06 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
2024-01-08 15:26 ` Harald Jörg
2024-01-08 19:27 ` Stefan Kangas
2024-01-09 16:06 ` john muhl via Bug reports for GNU Emacs, the Swiss army knife of text editors [this message]
2024-01-09 18:23 ` bug#17948: 24.3; cperl: Needs an equivalent to `c-file-style' 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=87bk9uwkxi.fsf_-_@pub.pink \
--to=bug-gnu-emacs@gnu.org \
--cc=17948@debbugs.gnu.org \
--cc=eliz@gnu.org \
--cc=haj@posteo.de \
--cc=jm@pub.pink \
--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).