Title: Mimic SequenceInputStream interface more in SequenceMessageBufferInput by frsyuki · Pull Request #389 · msgpack/msgpack-java · GitHub
Open Graph Title: Mimic SequenceInputStream interface more in SequenceMessageBufferInput by frsyuki · Pull Request #389 · msgpack/msgpack-java
X Title: Mimic SequenceInputStream interface more in SequenceMessageBufferInput by frsyuki · Pull Request #389 · msgpack/msgpack-java
Description: I commented as #387 (review) but I changed my mind: I think it's ok to keep SequenceMessageBufferInput as a mimic of SequenceInputStream. We can create another class if necessary. A reason is that behavior of SequenceMessageBufferInput.close (which is same withSequenceInputStream.close) is surprising, in my opinion. I expect that it closes input sequence without reading buffers. But they consume all of them. However, change of the behavior is also surprising because its behavior becomes inconsistent from SequenceInputStream.
Open Graph Description: I commented as #387 (review) but I changed my mind: I think it's ok to keep SequenceMessageBufferInput as a mimic of SequenceInputStream. We can create another class if necessary. A reason is t...
X Description: I commented as #387 (review) but I changed my mind: I think it's ok to keep SequenceMessageBufferInput as a mimic of SequenceInputStream. We can create another class if necessary. A reason ...
Opengraph URL: https://github.com/msgpack/msgpack-java/pull/389
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:122903e2-f846-22cb-ad8d-25f563ebbff4 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | E390:319EEE:1C02DE:273CCF:6A551419 |
| html-safe-nonce | 26c64ca14e00d057758dc10baa1c88be7d9a1e2090cc3ba58a2d7bad1595d319 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFMzkwOjMxOUVFRToxQzAyREU6MjczQ0NGOjZBNTUxNDE5IiwidmlzaXRvcl9pZCI6IjYxMDk5NTE1ODE1MDMxNjU0NjUiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 2ad47ee434344b7042d86aae2977097fbc2f65bb2f8147bd23d80ecafce3c185 |
| hovercard-subject-tag | pull_request:86560745 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/msgpack/msgpack-java/pull/389/files |
| twitter:image | https://avatars.githubusercontent.com/u/40720?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/40720?s=400&v=4 |
| og:image:alt | I commented as #387 (review) but I changed my mind: I think it's ok to keep SequenceMessageBufferInput as a mimic of SequenceInputStream. We can create another class if necessary. A reason is t... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 682d273eacb2ac51680c6eb9c0b270f029f7ce74c32090f319083c34497e28a5 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/msgpack/msgpack-java git https://github.com/msgpack/msgpack-java.git |
| octolytics-dimension-user_id | 198264 |
| octolytics-dimension-user_login | msgpack |
| octolytics-dimension-repository_id | 1971346 |
| octolytics-dimension-repository_nwo | msgpack/msgpack-java |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1971346 |
| octolytics-dimension-repository_network_root_nwo | msgpack/msgpack-java |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 3b6d37c6470adadff4194742daaab9a817cc4980 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width