[LU-6508] llog_test.c::llog_test_8 unnecessary use of rc2

• Type: Bug
• Resolution: Unresolved
• Priority: Minor
• Labels: easy tests
• Severity: 3
• Assignee: WC Triage
• Reporter: Oleg Drokin

Description

It looks like there are a couple of spots in llog_test_8 that use rc2 redundantly sine rc would always be 0 in those cases:

/home/green/smt/git/lustre-release/lustre/obdclass/llog_test.c:995 llog_test_8() warn: we tested 'rc' before and it was 'false'
/home/green/smt/git/lustre-release/lustre/obdclass/llog_test.c:1026 llog_test_8() warn: we tested 'rc' before and it was 'false'

I feel like if we take them out and replace with just plain rc assignment, the code would become a tiny bit more readable.