Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ydah
Copy link
Member

@ydah ydah commented Jun 9, 2024

This PR improve --report=terms command.

Output Unused Terms only when there are unused terms.
Also, output the number of terms as follows:

Before:

 Unused Terms
     0 YYerror
     2 YYUNDEF
     3 '\\\\'
     3 '\\13'
     4 keyword_class2
     5 tNUMBER
     6 tPLUS
     7 tMINUS
     8 tEQ
     9 tEQEQ
    10 '>'

After:

 11 Unused Terms
     0 YYerror
     1 YYUNDEF
     2 '\\\\'
     3 '\\13'
     4 keyword_class2
     5 tNUMBER
     6 tPLUS
     7 tMINUS
     8 tEQ
     9 tEQEQ
    10 '>'

ydah added 3 commits June 10, 2024 01:08
Before:
```
 Unused Terms
     0 YYerror
     2 YYUNDEF
     3 '\\\\'
     3 '\\13'
     4 keyword_class2
     5 tNUMBER
     6 tPLUS
     7 tMINUS
     8 tEQ
     9 tEQEQ
    10 '>'
```

After:
```
 11 Unused Terms
     0 YYerror
     1 YYUNDEF
     2 '\\\\'
     3 '\\13'
     4 keyword_class2
     5 tNUMBER
     6 tPLUS
     7 tMINUS
     8 tEQ
     9 tEQEQ
    10 '>'
```
@ydah ydah force-pushed the improve-report-unused-terms branch from 7c06a88 to fec1f15 Compare June 9, 2024 16:10
@ydah ydah merged commit 13140bc into ruby:master Jun 10, 2024
@ydah ydah deleted the improve-report-unused-terms branch June 10, 2024 07:29
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.

1 participant