Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New failures with native-compilation on emacs-next-minimal-30.0.50-1.9d27b95 #238

Closed
nicolas-graves opened this issue Sep 27, 2023 · 2 comments

Comments

@nicolas-graves
Copy link

`pretty' should pretty-print frames  Expected `(buttercup-output)' with value "========================================
suite bc-bt-backtrace

Traceback (most recent call last):
λ (bc-bt-foo
   \"a string that will be truncated in backtrace crop, at least 70 chars long\")
λ (bc-bt-bar
   \"a string that will be truncated in backtrace crop, at least 70 chars long\")
λ (bc-bt-baz
   \"a string that will be truncated in backtrace crop, at least 70 chars long\")
M (or (number-or-marker-p a)
      (signal 'wrong-type-argument (list 'number-or-marker-p a)))
λ (signal wrong-type-argument
        (number-or-marker-p
         \"a string that will be truncated in backtrace crop, at least 70 chars long\"))
error: (wrong-type-argument number-or-marker-p \"a string that will be truncated in backtrace crop, at least 70 chars long\")

" to match the regexp "\\(?:\\`=\\{40\\}
suite bc-bt-backtrace

Traceback (most recent call last):
λ (bc-bt-foo \"\\(?:a string that will be truncated in backtrace crop, at least 70 chars long\\)\")
λ (bc-bt-bar \"\\(?:a string that will be truncated in backtrace crop, at least 70 chars long\\)\")
λ (bc-bt-baz \"\\(?:a string that will be truncated in backtrace crop, at least 70 chars long\\)\")
\\(?:[Mλ] (.*\\(?:
   .*\\)*?.*)
\\)*error: ([^z-a]*)

\\'\\)", but instead it was "========================================
suite bc-bt-backtrace

Traceback (most recent call last):
λ (bc-bt-foo
   \"a string that will be truncated in backtrace crop, at least 70 chars long\")
λ (bc-bt-bar
   \"a string that will be truncated in backtrace crop, at least 70 chars long\")
λ (bc-bt-baz
   \"a string that will be truncated in backtrace crop, at least 70 chars long\")
M (or (number-or-marker-p a)
      (signal 'wrong-type-argument (list 'number-or-marker-p a)))
λ (signal wrong-type-argument
        (number-or-marker-p
         \"a string that will be truncated in backtrace crop, at least 70 chars long\"))
error: (wrong-type-argument number-or-marker-p \"a string that will be truncated in backtrace crop, at least 70 chars long\")

"

I don't see any difference by eye or with ediff. Maybe this has something to do with regexp or native compilation?

@mekeor
Copy link

mekeor commented Sep 30, 2023

I think it's because the regexp-pattern

λ (bc-bt-foo "\(?:a string that will be truncated in backtrace crop, at least 70 chars long\)")

does not match

λ (bc-bt-foo
   "a string that will be truncated in backtrace crop, at least 70 chars long")

because of the newline.

@snogge
Copy link
Collaborator

snogge commented Oct 18, 2023

Fixed in dfef8cf.
Don't know why I did not see your bug report or PR at the time. Must check my notification settings again.

@snogge snogge closed this as completed Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants