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

Rename data -> buffer for better readability. #7836

Merged
merged 1 commit into from
May 22, 2023

Conversation

ioquatix
Copy link
Member

No description provided.

@ioquatix ioquatix merged commit bcb16eb into ruby:master May 22, 2023
89 of 94 checks passed
@ioquatix ioquatix deleted the io-buffer-rename-data-buffer branch May 22, 2023 03:27
nagachika added a commit to nagachika/ruby that referenced this pull request May 16, 2024
…95740fed2a05a04806a1d3827bcaba02e45720: [Backport #19753]

	Adjust `else` style to be consistent in each files [ci skip]

	Rename `data` -> `buffer` for better readability. (ruby#7836)

	Fix `io_buffer_get_string` default length computation. (ruby#8427)

	* Fix `io_buffer_get_string` default length computation.

	When an offset bigger than the size is given, the resulting length will be
	computed incorrectly. Raise an argument error in this case.

	* Validate all arguments.
nagachika added a commit to nagachika/ruby that referenced this pull request May 16, 2024
…95740fed2a05a04806a1d3827bcaba02e45720: [Backport #19753]

	Adjust `else` style to be consistent in each files [ci skip]

	Rename `data` -> `buffer` for better readability. (ruby#7836)

	Fix `io_buffer_get_string` default length computation. (ruby#8427)

	* Fix `io_buffer_get_string` default length computation.

	When an offset bigger than the size is given, the resulting length will be
	computed incorrectly. Raise an argument error in this case.

	* Validate all arguments.
nagachika pushed a commit to nagachika/ruby that referenced this pull request May 16, 2024
nagachika pushed a commit that referenced this pull request May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant