Skip to content

Fix error logger crash#23

Open
Technologicat wants to merge 1 commit intoazazel75:masterfrom
Technologicat:errorloggerfix
Open

Fix error logger crash#23
Technologicat wants to merge 1 commit intoazazel75:masterfrom
Technologicat:errorloggerfix

Conversation

@Technologicat
Copy link

Avoid crashing in error logger when the message of a macro expansion failure is not a string.

This happens more often than one would think, when developing new macros :)

@Technologicat
Copy link
Author

Ping? This is a straightforward bugfix, so I'd like to get it into the official version of macropy3, if possible. :)

@azazel75
Copy link
Owner

It would be nice to have a testcase ;-)

@Technologicat
Copy link
Author

Now that I noticed the conflicts alert, it seems you already fixed this - in exactly the same way as I did - in 9cac7df. ;)

But I suppose I can provide a test before closing this.

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.

2 participants