jglick opened a new pull request, #740: URL: https://github.com/apache/commons-io/pull/740
Very trivial, but I noticed that the parameter of this method was named `idx`, which would usually be construed as an abbreviation for `index`, which in this context would be presumed to be an index into a `byte[]`. But that is not what it is; it is a byte (typed as `int` for what I suppose are historical reasons). The supertypes `OutputStream` and `FilterOutputStream` name the parameter `b`, which seems more intuitive. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@commons.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
