pilorama: Use AppendUint* helpers from stdlib #1525

Merged
fyrchik merged 1 commit from fyrchik/frostfs-node:patch-append into master 2024-11-28 07:49:54 +00:00
Owner

gopatch:

@@
var slice, e expression
@@
+import "encoding/binary"

-append(slice, byte(e), byte(e >> 8))
+binary.LittleEndian.AppendUint16(slice, e)

Signed-off-by: Evgenii Stratonikov e.stratonikov@yadro.com

gopatch: ``` @@ var slice, e expression @@ +import "encoding/binary" -append(slice, byte(e), byte(e >> 8)) +binary.LittleEndian.AppendUint16(slice, e) ``` Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
fyrchik added 1 commit 2024-11-28 06:16:45 +00:00
pilorama: Use AppendUint* helpers from stdlib
Some checks failed
Tests and linters / Run gofumpt (pull_request) Successful in 2m10s
DCO action / DCO (pull_request) Failing after 2m27s
Pre-commit hooks / Pre-commit (pull_request) Successful in 2m38s
Vulncheck / Vulncheck (pull_request) Successful in 2m49s
Build / Build Components (pull_request) Successful in 3m14s
Tests and linters / gopls check (pull_request) Successful in 3m28s
Tests and linters / Staticcheck (pull_request) Successful in 3m59s
Tests and linters / Lint (pull_request) Successful in 4m37s
Tests and linters / Tests (pull_request) Successful in 5m32s
Tests and linters / Tests with -race (pull_request) Successful in 5m57s
119c12f737
gopatch:
```
@@
var slice, e expression
@@
+import "encoding/binary"

-append(slice, byte(e), byte(e >> 8))
+binary.LittleEndian.AppendUint16(slice, e)
```

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
fyrchik requested review from storage-core-developers 2024-11-28 06:16:57 +00:00
fyrchik requested review from storage-core-committers 2024-11-28 06:17:00 +00:00
fyrchik force-pushed patch-append from 119c12f737 to 2dde5f90a1 2024-11-28 06:21:26 +00:00 Compare
acid-ant approved these changes 2024-11-28 06:29:35 +00:00
fyrchik force-pushed patch-append from 2dde5f90a1 to 01acec708f 2024-11-28 06:40:27 +00:00 Compare
elebedeva approved these changes 2024-11-28 07:36:07 +00:00
aarifullin approved these changes 2024-11-28 07:47:05 +00:00
fyrchik merged commit 01acec708f into master 2024-11-28 07:49:54 +00:00
fyrchik deleted branch patch-append 2024-11-28 07:49:57 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#1525
No description provided.