Skip to content

str.isprintable #1024

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 9, 2019
Merged

str.isprintable #1024

merged 1 commit into from
Jun 9, 2019

Conversation

yanganto
Copy link
Contributor

@yanganto yanganto commented Jun 9, 2019

Implement isprintable function of str object

@coolreader18 coolreader18 added the C-compat A discrepancy between RustPython and CPython label Jun 9, 2019
@codecov-io
Copy link

codecov-io commented Jun 9, 2019 •
edited
Loading

Codecov Report

Merging #1024 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1024      +/-   ##
==========================================
+ Coverage    64.7%   64.71%   +0.01%     
==========================================
  Files          97       97              
  Lines       17041    17047       +6     
  Branches     3799     3800       +1     
==========================================
+ Hits        11026    11032       +6     
  Misses       3441     3441              
  Partials     2574     2574
Impacted Files Coverage Δ
vm/src/obj/objstr.rs 74% <100%> (+0.21%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b968e9...f76be43. Read the comment docs.

@yanganto yanganto mentioned this pull request Jun 9, 2019
Closed
windelbouwman
windelbouwman reviewed Jun 9, 2019
View reviewed changes
windelbouwman
windelbouwman reviewed Jun 9, 2019
View reviewed changes
@windelbouwman
Copy link
Contributor

@yanganto thank you for contributing to this project! I left some review comments in your changes.

@yanganto yanganto force-pushed the str_isprintable branch 4 times, most recently from 53adcc9 to c8d45a1 Compare June 9, 2019 12:54
coolreader18
coolreader18 approved these changes Jun 9, 2019
View reviewed changes
str.isprintable
f76be43 
- check unicode type by unicode_categories
- rm redundant check of empty string
@coolreader18 coolreader18 removed the C-compat A discrepancy between RustPython and CPython label Jun 9, 2019
@coolreader18 coolreader18 merged commit 68011df into RustPython:master Jun 9, 2019
@yanganto yanganto mentioned this pull request Jun 11, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@windelbouwman windelbouwman windelbouwman left review comments

@coolreader18 coolreader18 coolreader18 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yanganto @codecov-io @windelbouwman @coolreader18