search

vermaseren / form

The FORM project for symbolic manipulation of very big expressions
GNU General Public License v3.0
1.16k stars 138 forks source link

Fix freeze in parsing complex conjugate operator #502

Closed tueda closed 6 months ago

tueda commented 7 months ago

The test case will come later (maybe after #501).

coveralls commented 7 months ago

Coverage Status

coverage: 48.707% (-0.006%) from 48.713% when pulling 5df4429acd9896772e3b34516f5d47cbd6355c39 on tueda:fix-conj-freeze into d55f894ffc5b5806d9fac21f01f1ae22ba215c8c on vermaseren:master.

tueda commented 6 months ago

In this version, the compiler, rather than the parser, generates the error. I also shortened the error message (which fits in one line, so the merged PR #504 was not mandatory for this PR at the end of the day) and added test cases.

@jodavies What do you think?

jodavies commented 6 months ago

This looks fine to me, yes.

tueda commented 6 months ago

@jodavies Thanks. I will merge it.