Buffer.alloc with non-buffer fill
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 35/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 停滞
- 技术栈
- javascript, nodejs
- 领域
- web-dev
调研方向
首先将 Buffer.alloc(size[, fill[, encoding]]) 与 Node 当前的 buffer 实现和测试进行比较，尤其是此处所示的 Uint8Array fill 情况。查看 #177 中的相关讨论，并确定哪些 Node buffer 测试与此相关；完成的标准是保留从 0 到 255 的 Uint8Array 字节，并通过测试覆盖该行为。
由索引模型根据 Issue 内容生成。
描述
Hi @feross!
If you pass a <Uint8Array> as the fill to Buffer.alloc(size[, fill[, encoding]]) node (v14.15.1) will copy those bytes verbatim into the buffer, whereas with the ferros/buffer package, it does something and the end result is that the resulting buffer is filled with strange data.
const input = [ 247, 108, 225 ]
const typedArr = new Uint8Array(input)
const output = Array.from(Buffer.alloc(typedArr.byteLength, typedArr))
// node:
// [ 247, 108, 225 ] (matches input, expected)
// feross/buffer:
// [ 239, 191, 189 ] (???)
It works fine for numbers between 0 and 127, but things get weird once you're out of those bounds (Uint8 should work for numbers between 0 - 255).
Workaround is to pass a Buffer as the fill value:
const output = Array.from(Buffer.alloc(typedArr.byteLength, Buffer.from(typedArr)))
// node:
// [ 247, 108, 225 ] (cool)
// feross/buffer:
// [ 247, 108, 225 ] (cool)
I'd perhaps PR a fix for this specific issue, but I feel like the real problem is that this package needs to be refreshed with the latest buffer code + tests from node, rather than me patching in a specific fix? i.e. Perhaps related to #177. I looked into this briefly and the transform from node looked non-trivial if you haven't done it before.
- 主要语言
- JavaScript
- 星标
- 1.9k
- 派生
- 260
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
我们还没有检查这个项目的环境配置文件。先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
feross/buffer 的其他 Issue
-
难度 4/5 3-5 天 新手友好度 25/100
-
Use `DataView`, remove dependency on `ieee754`可能重新可做 @dcousens 于 626 天前认领，目前没有进行中的 PR。 未关闭
-
难度 4/5 3-5 天 新手友好度 35/100
-
难度 2/5 1-3 小时 新手友好度 48/100
-
难度 3/5 1-2 天 新手友好度 38/100
相似的 Issue
-
bug good first issue
难度 2/5 1-3 小时 新手友好度 86/100
amponce/archive-movie-browser#354 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 74/100
saayam-for-all/webapp#1870 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 76/100
Imageomics/OpenCite#66 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 64/100
chr15m/twiiit.com#20 ·
-
documentation
难度 2/5 1-3 小时 新手友好度 88/100
antropos17/Aegis#629 ·
维护者通常 4 天内回复