Skip to content

mount.isFUSE(): remove unused error return, and extract FUSE unmount to a function#4588

Merged
dmcgowan merged 2 commits intocontainerd:masterfrom
thaJeztah:remove_redundant_error
Oct 8, 2020
Merged

mount.isFUSE(): remove unused error return, and extract FUSE unmount to a function#4588
dmcgowan merged 2 commits intocontainerd:masterfrom
thaJeztah:remove_redundant_error

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

This function was added in #3765; the error itself was unused, so may as well remove it.

@thaJeztah
Copy link
Copy Markdown
Member Author

@AkihiroSuda ptal

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 28, 2020

Build succeeded.

@thaJeztah thaJeztah force-pushed the remove_redundant_error branch from 316d9fa to 6081ba7 Compare September 28, 2020 11:52
@thaJeztah thaJeztah changed the title mount.isFUSE(): remove unused error return mount.isFUSE(): remove unused error return, and extract FUSE unmount to a function Sep 28, 2020
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 28, 2020

Build succeeded.

Comment thread mount/mount_linux.go Outdated
Comment thread mount/mount_linux.go Outdated
Comment thread mount/mount_linux.go Outdated
The error itself was unused, so may as well remove it.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah force-pushed the remove_redundant_error branch from 6081ba7 to 968a386 Compare September 28, 2020 19:47
@thaJeztah
Copy link
Copy Markdown
Member Author

@AkihiroSuda updated; I think this is what you meant; ptal

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 28, 2020

Build succeeded.

@thaJeztah thaJeztah force-pushed the remove_redundant_error branch 2 times, most recently from 0eb3105 to 9f1356f Compare October 1, 2020 15:20
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Oct 1, 2020

Build succeeded.

@thaJeztah thaJeztah force-pushed the remove_redundant_error branch from 9f1356f to 48f64a1 Compare October 1, 2020 15:29
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Oct 1, 2020

Build succeeded.

Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@dmcgowan dmcgowan merged commit 45d8a7e into containerd:master Oct 8, 2020
@thaJeztah thaJeztah deleted the remove_redundant_error branch October 8, 2020 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants