[haiku-commits] haiku: hrev50761 - src/kits/storage

  • From: jscipione@xxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Mon, 12 Dec 2016 19:40:27 +0100 (CET)

hrev50761 adds 1 changeset to branch 'master'
old head: 5967215c7b488a6282c6f5fff0d690b4d4288660
new head: 3db319329006e7c66e6a26796e6eb7a1b80b5f80
overview: 
http://cgit.haiku-os.org/haiku/log/?qt=range&q=3db319329006+%5E5967215c7b48

----------------------------------------------------------------------------

3db319329006: BNode: CID 602323 explicitly ignore fCStatus return value in ctor
  
  This is what it this whole push was all about

                                     [ John Scipione <jscipione@xxxxxxxxx> ]

----------------------------------------------------------------------------

Revision:    hrev50761
Commit:      3db319329006e7c66e6a26796e6eb7a1b80b5f80
URL:         http://cgit.haiku-os.org/haiku/commit/?id=3db319329006
Author:      John Scipione <jscipione@xxxxxxxxx>
Date:        Wed Dec  7 22:14:14 2016 UTC

----------------------------------------------------------------------------

1 file changed, 8 insertions(+), 4 deletions(-)
src/kits/storage/Node.cpp | 12 ++++++++----

----------------------------------------------------------------------------

diff --git a/src/kits/storage/Node.cpp b/src/kits/storage/Node.cpp
index 5ecc21f..c04de67 100644
--- a/src/kits/storage/Node.cpp
+++ b/src/kits/storage/Node.cpp
@@ -108,7 +108,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);
 }
 
 
@@ -118,7 +119,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);
 }
 
 
@@ -128,7 +130,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);
 }
 
 
@@ -138,7 +141,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);
 }
 
 


Other related posts: