diff options
author | Jim Meyering <meyering@redhat.com> | 2010-02-10 11:37:23 +0100 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2010-02-10 11:38:56 +0100 |
commit | 9f735dd16ab54550329f2fad357d816f7438e7d0 (patch) | |
tree | 45f55292ff042aff8b8fe39b240c4ee2037dd504 /src/analyze.c | |
parent | b50a16567acff496bdf3ac0004809b6dcc88c2d6 (diff) | |
download | diffutils-9f735dd16ab54550329f2fad357d816f7438e7d0.tar.gz |
maint: change use of "|" to more maintainable "||" (no semantic change)
* src/analyze.c (diff_2_files): Using the "||" operator happens to
be equivalent to using "|" in this case. It is also clearer and
less prone to inadvertent bug introduction, in case the variable,
"changes" were ever to take on a value not in {0,1}.
Patch by Tim Waugh, via Mike Frysinger.
Diffstat (limited to 'src/analyze.c')
-rw-r--r-- | src/analyze.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/analyze.c b/src/analyze.c index 1288ea0..0a18334 100644 --- a/src/analyze.c +++ b/src/analyze.c @@ -633,7 +633,7 @@ diff_2_files (struct comparison *cmp) changes = briefly_report (changes, cmp->file); else { - if (changes | !no_diff_means_no_output) + if (changes || !no_diff_means_no_output) { /* Record info for starting up output, to be used if and when we have some output to print. */ |