-
Notifications
You must be signed in to change notification settings - Fork 233
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
Minor Folding improvements #700
Comments
You see our activities for a couple of improvements in ical.net. Is there a chance you gave us a helping hand? |
Now you found the best iCalendar library for .Net 👍 |
It seems so.
I could try.
As I notice, it's another place for improvement: What do you say about it?
I need more time to explore all of the library to figure out its interconnections and probably to finish with reading RFC-5545 to be more precise. |
@minichma and I are working on ical.net since October, so we're well aware of a certain learning curve. |
You beat me to it by literally a few hours.
So I looked at the resulting code and suggest a few small improvements to get it a bit more readable.
It's a pitty that we couldn't use
ReadOnlySpan<>
, but for now it's fine.Also I noticed wrong array length
76
instead of75
.You see it in a moment in my PR, that follows.
The text was updated successfully, but these errors were encountered: