-
Notifications
You must be signed in to change notification settings - Fork 866
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
Fix typos in LowLevelInfo #1914
Conversation
docs/LowLevelInfo.md
Outdated
the SRT source code describing some cross-source analysis that would be | ||
unobvious for a source code reviewer. It's not a complete documentation of | ||
non-obvious for a source code reviewer. It's not a complete documentation of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure about this change: "unobvious" -> "non-obvious". @stevomatthews what do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"that would not be obvious for a source code reviewer" is the safe way to go! ;-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@quink-black Please also apply this suggestion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. It's amazing I can learning English from this project : P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor suggestions
Co-authored-by: stevomatthews <smatthews@haivision.com>
No description provided.