Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Remove unused parameter 'name' from DataRowMessage. #573

merged 1 commit into from Apr 22, 2014


2 participants

This hasn't been a problem (at least not yet) because the length and fieldCount properties aren't used anywhere. Nevertheless, I reckoned this is the intended form of DataRowMessage since there is an invocation of "new DataRowMessage(length, fieldCount)" down the line.


brianc commented Apr 22, 2014

Nice catch! 🌅

@brianc brianc added a commit that referenced this pull request Apr 22, 2014

@brianc brianc Merge pull request #573 from karvaporo/master
Remove unused parameter 'name' from DataRowMessage.

@brianc brianc merged commit 65d177e into brianc:master Apr 22, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment