[PATCH rtems-tools v1 4/4] ObjdumpProcessor.h: Remove inputBuffer_m

Gedare Bloom gedare at rtems.org
Wed Sep 1 20:15:16 UTC 2021


On Wed, Sep 1, 2021 at 2:00 PM Ryan Long <ryan.long at oarcorp.com> wrote:
>
> Accidentally got left in, so it is being removed.
>
Change the commit message to say you're removing it because it is
unused. If you want to refer when it accidentally got left in, you can
include the commit hash, but it's not necessary to do so.

Otherwise, the patch series looks good to me.

> CID 1506210: Uninitialized pointer field
>
> Closes #4492
> ---
>  tester/covoar/ObjdumpProcessor.h | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/tester/covoar/ObjdumpProcessor.h b/tester/covoar/ObjdumpProcessor.h
> index ed36981..6a207dd 100644
> --- a/tester/covoar/ObjdumpProcessor.h
> +++ b/tester/covoar/ObjdumpProcessor.h
> @@ -184,11 +184,6 @@ namespace Coverage {
>      );
>
>      /*!
> -     * This member variable is a buffer for input
> -     */
> -    char* inputBuffer_m;
> -
> -    /*!
>       * This member variable contains the symbols to be analyzed
>       */
>      DesiredSymbols& symbolsToAnalyze_m;
> --
> 1.8.3.1
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list