Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(data_structures): add CodeBuffer::print_bytes_iter_unchecked method #9337

Conversation

overlookmotel
Copy link
Contributor

@overlookmotel overlookmotel commented Feb 24, 2025

Add CodeBuffer::print_bytes_iter_unchecked which is useful for pushing bytes from an iterator in an efficient manner. This is needed by ESTree serializer, but is also a useful API addition in itself.

Copy link
Contributor Author

overlookmotel commented Feb 24, 2025


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • 0-merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions bot added the C-enhancement Category - New feature or request label Feb 24, 2025
@overlookmotel overlookmotel force-pushed the 02-24-feat_data_structures_add_codebuffer_print_bytes_iter_unchecked_method branch from 8281028 to 6f5aae4 Compare February 24, 2025 17:25
@overlookmotel overlookmotel marked this pull request as ready for review February 24, 2025 17:27
Copy link

codspeed-hq bot commented Feb 24, 2025

CodSpeed Performance Report

Merging #9337 will not alter performance

Comparing 02-24-feat_data_structures_add_codebuffer_print_bytes_iter_unchecked_method (f21740e) with main (fc46218)

Summary

✅ 33 untouched benchmarks

@overlookmotel overlookmotel force-pushed the 02-24-feat_data_structures_add_codebuffer_print_bytes_iter_unchecked_method branch from 6f5aae4 to 33087dd Compare February 24, 2025 17:40
@overlookmotel overlookmotel force-pushed the 02-24-refactor_data_structures_add_debug_assertion_to_codebuffer_peek_nth_char_back_and_improve_safety_docs branch from 8c93db0 to a86b061 Compare February 24, 2025 17:40
@graphite-app graphite-app bot added the 0-merge Merge with Graphite Merge Queue label Feb 25, 2025
Copy link

graphite-app bot commented Feb 25, 2025

Merge activity

…ethod (#9337)

Add `CodeBuffer::print_bytes_iter_unchecked` which is useful for pushing bytes from an iterator in an efficient manner. This is needed by `ESTree` serializer, but is also a useful API addition in itself.
@graphite-app graphite-app bot force-pushed the 02-24-refactor_data_structures_add_debug_assertion_to_codebuffer_peek_nth_char_back_and_improve_safety_docs branch from a86b061 to 6a4e892 Compare February 25, 2025 04:39
@graphite-app graphite-app bot force-pushed the 02-24-feat_data_structures_add_codebuffer_print_bytes_iter_unchecked_method branch from 33087dd to f21740e Compare February 25, 2025 04:40
Base automatically changed from 02-24-refactor_data_structures_add_debug_assertion_to_codebuffer_peek_nth_char_back_and_improve_safety_docs to main February 25, 2025 04:46
@graphite-app graphite-app bot merged commit f21740e into main Feb 25, 2025
27 checks passed
@graphite-app graphite-app bot deleted the 02-24-feat_data_structures_add_codebuffer_print_bytes_iter_unchecked_method branch February 25, 2025 04:53
Boshen added a commit that referenced this pull request Feb 26, 2025
## [0.53.0] - 2025-02-26

- #9289
- 4a5a7cf napi/parser: [**BREAKING**] Remove magic string; enable utf16
span converter by default (#9291) (Boshen)

### Features

- 5c775ea ast/estree: Enable serialization without TS fields (#9285)
(overlookmotel)
- f21740e data_structures: Add `CodeBuffer::print_bytes_iter_unchecked`
method (#9337) (overlookmotel)
- e10fb97 ecmascript: Improve may_have_side_effects for `.length`
(#9366) (sapphi-red)
- 35e5ca9 ecmascript: Improve may_have_side_effects for `instanceof`
(#9365) (sapphi-red)
- 11012c6 ecmascript: Improve ValueType for coalesce operator (#9354)
(sapphi-red)
- b7998fd ecmascript: To_number for object without toString (#9353)
(sapphi-red)
- e51d563 minifier: Concatenate strings with template literals on right
side (#9356) (sapphi-red)
- 9d7db54 minifier: Concatenate strings with template literals (#9355)
(sapphi-red)
- 835ee95 wasm: Return estree with utf16 span offsets (#9376) (Boshen)

### Bug Fixes

- 6a8f53f ast/estree: Visit `JSXOpeningFragment` and
`JSXClosingFragment` (#9342) (overlookmotel)
- e303767 ast/estree: Fix ESTree AST for imports and exports (#9282)
(overlookmotel)
- 54d59f1 data_structures: Stack types correctly report allocation size
if allocation failure during grow (#9317) (overlookmotel)
- f5c8698 ecmascript: Correct may_have_side_effects for classes (#9367)
(sapphi-red)
- d3ed128 minifier: Do not remove `=== 0` if the lhs can be NaN (#9352)
(sapphi-red)

### Performance

- 82adab9 ast/estree: Speed up building UTF8-UTF16 translation table
with SIMD (#9359) (overlookmotel)
- 61939ca ast/estree: Faster UTF-8 to UTF-16 span conversion (#9349)
(overlookmotel)
- 1bfc459 ast/estree: Pre-allocate `CodeBuffer` for JSON output (#9340)
(overlookmotel)
- 018c523 ast/estree: `ESTree` serializer use `CodeBuffer` (#9331)
(overlookmotel)
- 35ee399 codegen: Use `iter::repeat_n` in `CodeBuffer` (#9325)
(overlookmotel)

### Documentation

- 8bd3e39 data_structures: Uppercase SAFETY comments (#9330)
(overlookmotel)

### Refactor

- d94fc15 allocator: Reduce scope of `unsafe` blocks (#9319)
(overlookmotel)
- 7427900 ast: Re-order `ExportDefaultDeclaration` fields (#9348)
(overlookmotel)
- b09249c ast/estree: Rename serializers and serialization methods
(#9284) (overlookmotel)
- 55ed1df ast/estree: Shorten `ESTree` impls for enums (#9275)
(overlookmotel)
- 9d98444 codegen, data_structures: Move `CodeBuffer` into
`oxc_data_structures` crate (#9326) (overlookmotel)
- 6a4e892 data_structures: Add debug assertion to
`CodeBuffer::peek_nth_char_back` and improve safety docs (#9336)
(overlookmotel)
- fc46218 data_structures: `CodeBuffer::print_str` use
`Vec::extend_from_slice` (#9332) (overlookmotel)
- 690bae5 data_structures: Stack types const assert `T` is not zero-size
type (#9318) (overlookmotel)
- 10ba2ea data_structures: Reduce scope of `unsafe` blocks (#9316)
(overlookmotel)
- beb8382 data_structures: `CodeBuffer::print_bytes_unchecked` take a
byte slice (#9327) (overlookmotel)
- faf966f ecmascript: Don't check side effects in constant_evaluation
(#9122) (sapphi-red)
- 2faabe1 estree: Make `itoa` dependency optional (#9338)
(overlookmotel)
- 4e9e8cf lexer: Reduce scope of `unsafe` blocks (#9320) (overlookmotel)
- c31b53f mangler: Reduce scope of `unsafe` blocks (#9321)
(overlookmotel)
- f10a6da mangler: Move base54 into seperate mod (#9278) (Cameron)
- 12e89e0 syntax: Reduce scope of `unsafe` blocks (#9322)
(overlookmotel)
- f39be5f traverse: Reduce scope of `unsafe` blocks (#9323)
(overlookmotel)

Co-authored-by: Boshen <1430279+Boshen@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0-merge Merge with Graphite Merge Queue C-enhancement Category - New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant