Skip to content

Add the ability to exclude specific separators from the parsing algor… #55

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aTTiny73
Copy link

@aTTiny73 aTTiny73 commented Apr 4, 2023

I have implemented a way for users to exclude specific separators from the parsing algorithm to make this package more versatile and flexible. Logic is based on storing excluded characters in a slice. While parsing the command if excluded characters appear they will be ignored. I have provided GET and SET methods for the excluded character's slice for easier interaction.

The changes include adding the following features:

- excludedSep field in the Parser struct to hold a list of separator characters to be ignored during parsing.
- SetExcludeSeparators and ExcludedSeparators methods in the Parser struct to respectively set and get the excluded separators.
- isExcluded method in the Parser struct to check if a separator should be ignored.

These changes enable the parser to handle cases where certain separators should be ignored during parsing.

Thank you for considering my contribution. Please let me know if there are any issues or concerns.

@mattn
Copy link
Owner

mattn commented Jul 16, 2023

Please add tests

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