From ede463195dee8536628c54ee6a4a888c0683a83f Mon Sep 17 00:00:00 2001 From: John Scipione Date: Wed, 7 Dec 2016 14:14:14 -0800 Subject: [PATCH] BNode: CID 602323 explicitly ignore fCStatus return value in ctor This is what it this whole push was all about --- src/kits/storage/Node.cpp | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/kits/storage/Node.cpp b/src/kits/storage/Node.cpp index 63f2a5b6c0..696567ccd1 100644 --- a/src/kits/storage/Node.cpp +++ b/src/kits/storage/Node.cpp @@ -109,7 +109,8 @@ BNode::BNode(const entry_ref* ref) fAttrFd(-1), fCStatus(B_NO_INIT) { - SetTo(ref); + // fCStatus is set by SetTo(), ignore return value + (void)SetTo(ref); } @@ -119,7 +120,8 @@ BNode::BNode(const BEntry* entry) fAttrFd(-1), fCStatus(B_NO_INIT) { - SetTo(entry); + // fCStatus is set by SetTo(), ignore return value + (void)SetTo(entry); } @@ -129,7 +131,8 @@ BNode::BNode(const char* path) fAttrFd(-1), fCStatus(B_NO_INIT) { - SetTo(path); + // fCStatus is set by SetTo(), ignore return value + (void)SetTo(path); } @@ -139,7 +142,8 @@ BNode::BNode(const BDirectory* dir, const char* path) fAttrFd(-1), fCStatus(B_NO_INIT) { - SetTo(dir, path); + // fCStatus is set by SetTo(), ignore return value + (void)SetTo(dir, path); }