diff options
| author |   Alex Elder <elder@dreamhost.com> | 2012-02-15 07:43:54 -0600 |
|---|---|---|
| committer | Alex Elder <elder@dreamhost.com> | 2012-03-22 10:47:51 -0500 |
| commit | bca064d236a2e3162a07c758855221bcbe3c475b (patch) | |
| tree | 49fca3de9007fa6cc5304bd8d7851d28cbe33110 /Documentation | |
| parent | ceph: ensure Boolean options support both senses (diff) | |
| download | linux-dev-bca064d236a2e3162a07c758855221bcbe3c475b.tar.xz linux-dev-bca064d236a2e3162a07c758855221bcbe3c475b.zip | |
libceph: use "do" in CRC-related Boolean variables
Change the name (and type) of a few CRC-related Boolean local
variables so they contain the word "do", to distingish their purpose
from variables used for holding an actual CRC value.
Note that in the process of doing this I identified a fairly serious
logic error in write_partial_msg_pages(): the value of "do_crc"
assigned appears to be the opposite of what it should be. No
attempt to fix this is made here; this change preserves the
erroneous behavior. The problem I found is documented here:
http://tracker.newdream.net/issues/2064
Signed-off-by: Alex Elder <elder@dreamhost.com>
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'Documentation')
0 files changed, 0 insertions, 0 deletions