diff options
| author | Sage Weil <sage@inktank.com> | 2013-10-10 07:10:16 -0700 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2013-10-11 09:23:10 -0700 |
| commit | 7113186c32c90713369a8d1a8e94d6a0ccba8394 (patch) | |
| tree | 75e26e033018b9f55116d29e4152b5a8714a7330 | |
| parent | 82e93303fb83e886206b3157b2fdc3d6617e9934 (diff) | |
| download | ceph-7113186c32c90713369a8d1a8e94d6a0ccba8394.tar.gz | |
osdc/Objecter: clean up completion handlers that set *prval=0
If the OSD operations returns 0, these are unnecessary. If it does not,
then these are misleading. In both cases here, the OSD sets the result to
zero, so this has no client-visible change.r
Signed-off-by: Sage Weil <sage@inktank.com>
| -rw-r--r-- | src/osdc/Objecter.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/osdc/Objecter.h b/src/osdc/Objecter.h index 8a6c243d339..938c97a4f31 100644 --- a/src/osdc/Objecter.h +++ b/src/osdc/Objecter.h @@ -386,7 +386,6 @@ struct ObjectOperation { pwatchers->push_back(ow); } } - *prval = 0; } catch (buffer::error& e) { if (prval) @@ -424,8 +423,6 @@ struct ObjectOperation { } psnaps->seq = resp.seq; } - if (prval) - *prval = 0; } catch (buffer::error& e) { if (prval) |