Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 34.2k
doc: added TC meeting minutes 2014-12-17#178
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
Conversation
rvagg commented Dec 17, 2014
Two outstanding items that need to be taken care of:
I'm noting them here because I don't have time to deal with them just yet! |
iankronquist commented Dec 18, 2014
roadmap++ |
bnoordhuis commented Dec 18, 2014
LGTM with the caveat that I wasn't there and haven't watched the video yet. :-) |
Closes#163 PR-URL: #178 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Bert Belder <[email protected]>
piscisaureus commented Dec 31, 2014
@rvagg I landed this PR - why reopen? |
rvagg commented Dec 31, 2014
@piscisaureus there's 2 items in the comments here that I still need to take care of, TODOs for me |
piscisaureus commented Dec 31, 2014
Ok got it |
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.
Blog of FileReader
Blob or FileReader?
The snprintf API is not available with older versions of MSVC. Found by a CI run.
Co-authored-by: François Doray <[email protected]>
Co-authored-by: François Doray <[email protected]>
Co-authored-by: François Doray <[email protected]>
Closes#163