commit | dcacdad4746429cc92c12a33eaf22f879408e47a | [log] [tgz] |
---|---|---|
author | Kir Kolyshkin <kolyshkin@gmail.com> | Wed May 18 15:26:56 2022 -0700 |
committer | Gopher Robot <gobot@golang.org> | Thu May 19 14:10:25 2022 +0000 |
tree | 88f54fb559513e8745f3c716d3f68e48c1129a60 | |
parent | 5e4e11fc645e56abf0a217a04090d2b6a036bf42 [diff] |
unix: use ByteSliceFromString in (*Ifreq).Name CL 340370 introduced (*Ifreq).Name method which scans the Ifrn field for \0 twice -- first explicitly, when by calling BytePtrToString. It seems more straightforward to use ByteSliceToString instead. The differences are: - simpler code; - no double scanning for \0; - in case there is no \0 in the Ifrn (unlikely), the full string (rather than an empty string) is now returned. With the last item in mind, the test case with no \0 as input that expects empty output fails, so drop it. Alternatively, we could test that it returns the full string, but it will be essentially testing ByteSliceToString, which is not the point. Cc: Matt Layher <mdlayher@gmail.com> Change-Id: I31d4f6dbe98aae5120f9b2246c93ceaa6a165395 Reviewed-on: https://go-review.googlesource.com/c/sys/+/407194 Reviewed-by: Ian Lance Taylor <iant@google.com> Auto-Submit: Ian Lance Taylor <iant@google.com> Reviewed-by: Matt Layher <mdlayher@gmail.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Matt Layher <mdlayher@gmail.com> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> Run-TryBot: Ian Lance Taylor <iant@google.com>
This repository holds supplemental Go packages for low-level interactions with the operating system.
The easiest way to install is to run go get -u golang.org/x/sys
. You can also manually git clone the repository to $GOPATH/src/golang.org/x/sys
.
This repository uses Gerrit for code changes. To learn how to submit changes to this repository, see https://golang.org/doc/contribute.html.
The main issue tracker for the sys repository is located at https://github.com/golang/go/issues. Prefix your issue with “x/sys:” in the subject line, so it is easy to find.