Coverity Issue with bdbuf.c

[Gedare Bloom]

I'm more concerned with the hard-coded cache alignment value, than I
am with the dead code.

-Gedare

On Wed, Nov 19, 2014 at 6:10 PM, Joel Sherrill
<joel.sherrill at oarcorp.com> wrote:
> Hi
>
> Coverity ID: 1255327
>
> I think this looks like a legitimately flagged piece of code :
>
> 1537   * For unspecified cache alignments we use the CPU alignment.
> 1538   */
>
> assignment: Assigning: cache_aligment = 32U.
> 1539  cache_aligment = 32; /* FIXME rtems_cache_get_data_line_size() */
>
> const: At condition cache_aligment <= 0U, the value of cache_aligment must
> be equal to 32.
>
> dead_error_condition: The condition cache_aligment <= 0U cannot be true.
> 1540  if (cache_aligment <= 0)
>
> CID 1255327 (#1 of 1): Logically dead code (DEADCODE)dead_error_line:
> Execution cannot reach this statement: cache_aligment = 8U;.
> 1541    cache_aligment = CPU_ALIGNMENT;
>
> Josh .. can you check if CodeSonar flagged this?
>
> --
> Joel Sherrill, Ph.D.             Director of Research & Development
> joel.sherrill at OARcorp.com        On-Line Applications Research
> Ask me about RTEMS: a free RTOS  Huntsville AL 35805
> Support Available                (256) 722-9985
>
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel