Skip to content

Fixing reader creation logic in examples#21

Merged
satyrius merged 1 commit intosatyrius:masterfrom
mccv:mccv/fix_example_readers
Oct 30, 2015
Merged

Fixing reader creation logic in examples#21
satyrius merged 1 commit intosatyrius:masterfrom
mccv:mccv/fix_example_readers

Conversation

@mccv
Copy link
Copy Markdown

@mccv mccv commented Oct 30, 2015

Previously the creation of an actual file used := to assign the file and err.
This shadowed the var declaration of file, leaving it uninitialized and causing
seg faults when the scanner tried to read data.

@satyrius
Copy link
Copy Markdown
Owner

Thank you for pull request! Could you drop binaries committed by accident? Then it will be merged to master.

Previously the creation of an actual file used := to assign the file and err.
This shadowed the var declaration of file, leaving it uninitialized and causing
seg faults when the scanner tried to read data.
@mccv mccv force-pushed the mccv/fix_example_readers branch from 1096689 to 7a235fa Compare October 30, 2015 16:20
@mccv
Copy link
Copy Markdown
Author

mccv commented Oct 30, 2015

ugh, sorry. Done.

satyrius added a commit that referenced this pull request Oct 30, 2015
Fixing reader creation logic in examples
@satyrius satyrius merged commit 0f2fd61 into satyrius:master Oct 30, 2015
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