Bug 93399 - Remove redundant NULL check
Summary: Remove redundant NULL check
Status: RESOLVED FIXED
Alias: None
Product: realmd
Classification: Unclassified
Component: adcli (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Stef Walter
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-12-16 09:59 UTC by Sumit Bose
Modified: 2015-12-17 08:12 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
seq_remove: remove redundant NULL check (545 bytes, patch)
2015-12-16 10:00 UTC, Sumit Bose
Details | Splinter Review

Description Sumit Bose 2015-12-16 09:59:52 UTC
The 'if (length)' check is seq_remove() is redundant because it is already asserted that length is not NULL.
Comment 1 Sumit Bose 2015-12-16 10:00:17 UTC
Created attachment 120540 [details] [review]
seq_remove: remove redundant NULL check
Comment 2 Stef Walter 2015-12-17 08:08:58 UTC
I misunderstood when we talked earlier. This is correct.
Comment 3 Stef Walter 2015-12-17 08:12:59 UTC
Attachment 120540 [details] pushed as 6747efc - seq_remove: remove redundant NULL check


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.