Skip to content

Commit

Permalink
Rename block decoder type parsing option.
Browse files Browse the repository at this point in the history
Now that only `parse` is public (i.e. not `fromSchema`), this makes more
sense.
  • Loading branch information
mtth committed Nov 9, 2015
1 parent d63b79a commit bd17318
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion doc
Submodule doc updated from 70fff6 to 63eb12
4 changes: 2 additions & 2 deletions lib/files.js
Expand Up @@ -124,7 +124,7 @@ function BlockDecoder(opts) {

this._type = null;
this._codecs = opts.codecs;
this._typeOpts = opts.typeOpts || {};
this._parseOpts = opts.parseOpts || {};
this._tap = new Tap(new Buffer(0));
this._blockTap = new Tap(new Buffer(0));
this._syncMarker = null;
Expand Down Expand Up @@ -175,7 +175,7 @@ BlockDecoder.prototype._decodeHeader = function () {

try {
var schema = JSON.parse(header.meta['avro.schema'].toString());
this._type = schemas.createType(schema, this._typeOpts);
this._type = parse(schema, this._parseOpts);
} catch (err) {
this.emit('error', err);
return;
Expand Down
3 changes: 2 additions & 1 deletion package.json
Expand Up @@ -27,7 +27,8 @@
},
"scripts": {
"test": "mocha --ui tdd --reporter dot",
"cover": "istanbul cover _mocha -- --ui tdd --reporter dot"
"cover": "istanbul cover _mocha -- --ui tdd --reporter dot",
"clean": "rm -rf coverage node_modules"
},
"devDependencies": {
"coveralls": "^2.11.4",
Expand Down

0 comments on commit bd17318

Please sign in to comment.