Skip to content

pkg/mount: use sort.Slice#36506

Merged
vdemeester merged 1 commit intomoby:masterfrom
kolyshkin:pkg-mount-slice
Mar 9, 2018
Merged

pkg/mount: use sort.Slice#36506
vdemeester merged 1 commit intomoby:masterfrom
kolyshkin:pkg-mount-slice

Conversation

@kolyshkin
Copy link
Copy Markdown
Contributor

Sorting by mount point length can be implemented in a more straightforward fashion since Go 1.8 introduced sort.Slice() with an ability to provide a less() function in place.

(this used to be part of #36091; singled out for easier review)

Sorting by mount point length can be implemented in a more
straightforward fashion since Go 1.8 introduced sort.Slice()
with an ability to provide a less() function in place.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Copy link
Copy Markdown
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@vieux vieux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 9, 2018

Codecov Report

Merging #36506 into master will increase coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master   #36506      +/-   ##
==========================================
+ Coverage   34.64%   34.67%   +0.03%     
==========================================
  Files         613      612       -1     
  Lines       45404    45400       -4     
==========================================
+ Hits        15730    15743      +13     
+ Misses      27614    27599      -15     
+ Partials     2060     2058       -2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants