Copy link
Copy Markdown
Contributor
Copy link
Copy Markdown
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks safe to me, I've only left a comment about a minor adjustment.
| /* turn off color */ | ||
| fg = -1; | ||
| bg = -1; | ||
| } else { |
Copy link
Copy Markdown
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The if (**str != ',') { check below can be removed because now **str is a digit.
remove now useless check for ,
Copy link
Copy Markdown
Contributor Author
thanks for review, removed that check