all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: john muhl via "Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
To: Daniel Brooks <db48x@db48x.net>
Cc: Eli Zaretskii <eliz@gnu.org>,
	66944@debbugs.gnu.org, stefankangas@gmail.com
Subject: bug#66944: 30.0.50; [PATCH] make calc parse fractions written using U+2044 FRACTION SLASH
Date: Sat, 27 Jan 2024 12:41:08 -0600	[thread overview]
Message-ID: <87bk96bovv.fsf@pub.pink> (raw)
In-Reply-To: <8734uiajsz.fsf@db48x.net> (Daniel Brooks's message of "Sat, 27 Jan 2024 07:16:12 -0800")

[-- Attachment #1: Type: text/plain, Size: 358 bytes --]

Daniel Brooks <db48x@db48x.net> writes:

> Eli Zaretskii <eliz@gnu.org> writes:
>
>> Thanks, I installed the changes on the master branch, and I'm
>> therefore closing this bug.
>
> Awesome. Thanks!
>
> db48x

The calc tests started failing today. Bisect says 77d9d05df87 is
the culprit. Log attached.

  cd ~/src/emacs/test
  make lisp/calc/calc-tests.log


[-- Attachment #2: calc-tests.log --]
[-- Type: text/plain, Size: 4008 bytes --]

Running 25 tests (2024-01-27 12:27:30-0600, selector `(not (or (tag :unstable) (tag :nativecomp)))')
   passed   1/25  calc-Math-integerp (0.000105 sec)
   passed   2/25  calc-bit-ops (0.052791 sec)
Building units table...
Building units table...done
   passed   3/25  calc-bug-23889 (0.012645 sec)
   passed   4/25  calc-business-days (0.002711 sec)
   passed   5/25  calc-calendar (0.000170 sec)
   passed   6/25  calc-choose (0.003441 sec)
   passed   7/25  calc-convert-units (0.006384 sec)
   passed   8/25  calc-defmath (0.000104 sec)
   passed   9/25  calc-digit-after-point (0.000116 sec)
   passed  10/25  calc-extract-units (0.000571 sec)
   passed  11/25  calc-format-radix (0.000098 sec)
   passed  12/25  calc-frac-input (0.000218 sec)
   passed  13/25  calc-gcd (0.000080 sec)
   passed  14/25  calc-imaginary-i (0.001161 sec)
   passed  15/25  calc-latex-input (0.002340 sec)
Test calc-math-read-preprocess-string backtrace:
  signal(ert-test-failed (((should (string= (concat "+/-*:-/*inf<=>=<=
  ert-fail(((should (string= (concat "+/-*:-/*inf<=>=<=>=μ(1:4)(1:2)(3
  (if (unwind-protect (setq value-1055 (apply fn-1053 args-1054)) (set
  (let (form-description-1057) (if (unwind-protect (setq value-1055 (a
  (let ((value-1055 'ert-form-evaluation-aborted-1056)) (let (form-des
  (let* ((fn-1053 #'string=) (args-1054 (condition-case err (list (con
  (closure (var-g t) nil (let* ((fn-1023 #'string=) (args-1024 (condit
  #f(compiled-function () #<bytecode -0x666deb319dd90b0>)()
  handler-bind-1(#f(compiled-function () #<bytecode -0x666deb319dd90b0
  ert--run-test-internal(#s(ert--test-execution-info :test #s(ert-test
  ert-run-test(#s(ert-test :name calc-math-read-preprocess-string :doc
  ert-run-or-rerun-test(#s(ert--stats :selector ... :tests ... :test-m
  ert-run-tests((not (or (tag :unstable) (tag :nativecomp))) #f(compil
  ert-run-tests-batch((not (or (tag :unstable) (tag :nativecomp))))
  ert-run-tests-batch-and-exit((not (or (tag :unstable) (tag :nativeco
  eval((ert-run-tests-batch-and-exit '(not (or (tag :unstable) (tag :n
  command-line-1(("-L" ":." "-l" "ert" "-l" "lisp/calc/calc-tests.el" 
  command-line()
  normal-top-level()
Test calc-math-read-preprocess-string condition:
    (ert-test-failed
     ((should
       (string=
	(concat "+/-*:-/*inf<=>=<=>=μ(1:4)(1:2)(3:4)(1:3)(2:3)"
		"(1:5)(2:5)(3:5)(4:5)(1:6)(5:6)"
		"(1:8)(3:8)(5:8)(7:8)1:^(0123456789+-()ni)"
		"_(0123456789+-())")
	(math-read-preprocess-string ...)))
      :form
      (string=
       "+/-*:-/*inf<=>=<=>=μ(1:4)(1:2)(3:4)(1:3)(2:3)(1:5)(2:5)(3:5)(4:5)(1:6)(5:6)(1:8)(3:8)(5:8)(7:8)1:^(0123456789+-()ni)_(0123456789+-())"
       "+/-*:-/*inf<=>=<=>=μ(1:4)(1:2)(3:4)(1:3)(2:3)(1:5)(2:5)(3:5)(4:5)(1:6)(5:6)(1:8)(3:8)(5:8)(7:8)1::^(0123456789+-()ni)_(0123456789+-())")
      :value nil :explanation
      (arrays-of-different-length 133 134
				  "+/-*:-/*inf<=>=<=>=μ(1:4)(1:2)(3:4)(1:3)(2:3)(1:5)(2:5)(3:5)(4:5)(1:6)(5:6)(1:8)(3:8)(5:8)(7:8)1:^(0123456789+-()ni)_(0123456789+-())"
				  "+/-*:-/*inf<=>=<=>=μ(1:4)(1:2)(3:4)(1:3)(2:3)(1:5)(2:5)(3:5)(4:5)(1:6)(5:6)(1:8)(3:8)(5:8)(7:8)1::^(0123456789+-()ni)_(0123456789+-())"
				  first-mismatch-at 97)))
   FAILED  16/25  calc-math-read-preprocess-string (0.000212 sec) at lisp/calc/calc-tests.el:846
   passed  17/25  calc-matrix-determinant (0.003859 sec)
   passed  18/25  calc-nth-root (0.000213 sec)
Working...
(Remainder was 8 * x + 1)
   passed  19/25  calc-poly-div (0.002111 sec)
Working...

   passed  20/25  calc-remove-units (0.000210 sec)
   passed  21/25  calc-shift-binary (0.032695 sec)
   passed  22/25  calc-solve-linear-system (0.004019 sec)
   passed  23/25  calc-sum-gcd (0.000234 sec)
   passed  24/25  calc-trig (0.000546 sec)
   passed  25/25  calc-unix-date (0.000296 sec)

Ran 25 tests, 24 results as expected, 1 unexpected (2024-01-27 12:27:30-0600, 0.255913 sec)

1 unexpected results:
   FAILED  calc-math-read-preprocess-string


  reply	other threads:[~2024-01-27 18:41 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-05  8:18 bug#66944: 30.0.50; [PATCH] make calc parse fractions written using U+2044 FRACTION SLASH Daniel Brooks
2024-01-13 10:52 ` Stefan Kangas
2024-01-13 16:11   ` Daniel Brooks
2024-01-27  9:09     ` Eli Zaretskii
2024-01-27 13:11       ` Daniel Brooks
2024-01-27 14:00         ` Eli Zaretskii
2024-01-27 14:26           ` Daniel Brooks
2024-01-27 14:49             ` Eli Zaretskii
2024-01-27 15:16               ` Daniel Brooks
2024-01-27 18:41                 ` john muhl via Bug reports for GNU Emacs, the Swiss army knife of text editors [this message]
2024-01-28  8:20                   ` Daniel Brooks
2024-01-29 12:51                     ` Eli Zaretskii

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87bk96bovv.fsf@pub.pink \
    --to=bug-gnu-emacs@gnu.org \
    --cc=66944@debbugs.gnu.org \
    --cc=db48x@db48x.net \
    --cc=eliz@gnu.org \
    --cc=jm@pub.pink \
    --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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.