Skip to content

preserve multiline log messages#3975

Open
pkienzle wants to merge 1 commit into
mainfrom
3219-multiline-log-messages
Open

preserve multiline log messages#3975
pkienzle wants to merge 1 commit into
mainfrom
3219-multiline-log-messages

Conversation

@pkienzle

@pkienzle pkienzle commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Description

Preserve line breaks in error messages displayed in the log explorer.

Cleans up uncaught exception handling so that it appears in one place.

Minor unrelated code cleanup, removing an unused variable.

Fixes #3219

How Has This Been Tested?

Introduced an uncaught exception after plotting in the fitting widget so that uncaught exceptions could easily be generated (not included in the PR).

Created a plugin model with a c error so that compiler output could be checked.

Review Checklist:

[if using the editor, use [x] in place of [ ] to check a box]

Documentation (check at least one)

  • There is nothing that needs documenting
  • Documentation changes are in this PR
  • There is an issue open for the documentation (link?)

Installers

  • There is a chance this will affect the installers, if so
    • Windows installer (GH artifact) has been tested (installed and worked)
    • MacOSX installer (GH artifact) has been tested (installed and worked)
    • Wheels installer (GH artifact) has been tested (installed and worked)

Licensing (untick if necessary)

  • The introduced changes comply with SasView license (BSD 3-Clause)

@krzywon krzywon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine. Haven't tested.

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 this pull request may close these issues.

New lines don't work properly on the log explorer

2 participants