Opened 3 years ago
Last modified 3 years ago
#3145 accepted enhancement
Reduce the number of parentheses produced by the expression formatter for infix forms
| Reported by: | gkronber | Owned by: | gkronber |
|---|---|---|---|
| Priority: | medium | Milestone: | HeuristicLab 3.3.17 |
| Component: | Problems.DataAnalysis.Symbolic | Version: | trunk |
| Keywords: | Cc: |
Description (last modified by gkronber)
The formatter produces unnecessary parentheses for some expressions which makes the output harder to read than necessary.
Change History (7)
comment:1 Changed 3 years ago by gkronber
- Owner set to gkronber
- Status changed from new to accepted
comment:2 Changed 3 years ago by gkronber
- Description modified (diff)
- Summary changed from Reduce the number of parantheses produced by the expression formatter for infix forms to Reduce the number of parentheses produced by the expression formatter for infix forms
comment:3 Changed 3 years ago by gkronber
comment:4 Changed 3 years ago by gkronber
r18172: removed unnecessary parenthesis in InfixExpressionFormatter (and adapted the unit tests slightly)
comment:5 Changed 3 years ago by gkronber
r18203: fixed a bug in the infix formatter introduced in my earlier commit
comment:6 Changed 3 years ago by gkronber
r18211: refactored infix formatter to improve output (less parenthesis) and added unit tests to test that the infix formatter works correclty.
comment:7 Changed 3 years ago by gkronber
Similar changes should be applied to the LaTeX formatter.
Note: See
TracTickets for help on using
tickets.
r18170: changed unit tests to check new expected output (unnecessary parenthesis removed) --> unit tests fail