Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added missing translation of \f and \b #2135

Merged
merged 2 commits into from
Jan 29, 2025
Merged

Added missing translation of \f and \b #2135

merged 2 commits into from
Jan 29, 2025

Conversation

PieterOlivier
Copy link
Contributor

@PieterOlivier PieterOlivier commented Jan 29, 2025

Formfeed and backspace support was missing from char2int and from String::deescape

Copy link

codecov bot commented Jan 29, 2025

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 50%. Comparing base (27c2891) to head (78117e8).
Report is 11 commits behind head on main.

Files with missing lines Patch % Lines
...org/rascalmpl/values/parsetrees/SymbolFactory.java 0% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##              main   #2135    +/-   ##
========================================
  Coverage       49%     50%            
+ Complexity    6251    6248     -3     
========================================
  Files          675     673     -2     
  Lines        59270   58832   -438     
  Branches      8642    8599    -43     
========================================
- Hits         29579   29577     -2     
+ Misses       27458   27013   -445     
- Partials      2233    2242     +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@PieterOlivier PieterOlivier changed the title Added missing translation of \f to formfeed character Added missing translation of \f and \b Jan 29, 2025
@PieterOlivier PieterOlivier merged commit 4c2060a into main Jan 29, 2025
6 of 7 checks passed
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