Re: [PATCH] Remove useless typedefs

From: Kinkie <gkinkie_at_gmail.com>
Date: Thu, 1 Dec 2011 07:56:47 +0100

On Thu, Dec 1, 2011 at 1:12 AM, Amos Jeffries <squid3_at_treenet.co.nz> wrote:
> On Wed, 30 Nov 2011 18:11:52 +0100, Kinkie wrote:
>>
>> Hi all,
>>  I have noticed that typedefs.h contains a few typedefs of structs
>> which do not seem to be defined anywhere, and are not used anywhere.
>> This patch removes them.
>
>
> +1. Please commit useless code removals from typedefs.h, protos.h, structs.h
> on discovery.

Committed.

>> A different case is storeSwapLogData, which is used in
>> src/test_cache_digest.cc, and whose removal doesn't fail the build
>> test for some reason. I haven't removed it in the attached patch.
>
>
> That would be because the test_cache_digest is never actually run/tested :(
>  that needs a fix.
>
> The typedef appears could be moved to the tests .cc file.

Not even that, because it typedefs a struct which doesn't exist; I
would suggest to remove the test file as well.

-- 
    /kinkie
Received on Thu Dec 01 2011 - 06:56:54 MST

This archive was generated by hypermail 2.2.0 : Thu Dec 01 2011 - 12:00:10 MST