fix: addressing reviews · vibhatha/arrow@f083330

Test: build/js/src/util/buffer.ts#L134

Move generator function 'wrap' to the outer scope

Test: build/js/src/util/buffer.ts#L172

Move async generator function 'wrap' to the outer scope

Test: build/js/test/generate-test-data.ts#L325

Unnecessarily cloning an array

Test: build/js/test/generate-test-data.ts#L347

Unnecessarily cloning an array

Test: build/js/test/generate-test-data.ts#L368

Unnecessarily cloning an array

Test: build/js/test/generate-test-data.ts#L378

Unnecessarily cloning an array

Test: build/js/test/generate-test-data.ts#L486

Unnecessarily cloning an array

Test: build/js/test/generate-test-data.ts#L624

Unnecessarily cloning an array

Build: build/js/src/util/buffer.ts#L134

Move generator function 'wrap' to the outer scope

Build: build/js/src/util/buffer.ts#L172

Move async generator function 'wrap' to the outer scope

Build: build/js/test/generate-test-data.ts#L325

Unnecessarily cloning an array

Build: build/js/test/generate-test-data.ts#L347

Unnecessarily cloning an array

Build: build/js/test/generate-test-data.ts#L368

Unnecessarily cloning an array

Build: build/js/test/generate-test-data.ts#L378

Unnecessarily cloning an array

Build: build/js/test/generate-test-data.ts#L486

Unnecessarily cloning an array

Build: build/js/test/generate-test-data.ts#L624

Unnecessarily cloning an array

Provide feedback