Details
-
Sub-task
-
Resolution: Unresolved
-
Minor
-
None
-
None
Description
Ftr, the 2 * 3 previous cases are consistent.
This 7th case is warned:
256 /* Parse maximum view length per disk, cannot be smaller than 16MB */ |
257 if (MaximumPerDiskViewLength >= 0x1000000) |
258 {
|
259 if (MaxViewLength > 0xFFFFFFFF) MaximumPerDiskViewLength = -1; |
260 }
|
261 else |
262 {
|
263 MaximumPerDiskViewLength = 0x1000000u;
|
264 }
|
Line 259 is undocumented.
MaximumPerDiskViewLength grep is set, but never used.
Options:
- Remove MaximumPerDiskViewLength.
Assumed unwanted? - Convert this line to a // FIXME.
As this line is currently useless anyway! - Replace it with if (MaximumPerDiskViewLength > MaxViewLength) MaximumPerDiskViewLength = MaxViewLength;.
Only a guess...
This code was added as is in r34170 by "ros-arm-bringup".
—
While there, add u where they miss, to be consistent.
Ideally, use some {{#define}}s.