Comment 63 for bug 438136

Looking at atasmart.c in libatasmart-0.17:
is this the section that causes the warning:
starting at line 1269
--------
 if ((!strcmp(a->name, "reallocated-sector-count") ||
                     !strcmp(a->name, "current-pending-sector")) &&
                    a->pretty_value > 0)
                        a->warn = TRUE;
-------
What is the logic behind this?
Wouldn't the OR part always be true?
it is kind of like using (( x != 1 ) || ( x != 2 ))
this would be true with x equaling any number.