While it was added there (it was part of builtin pickaxe which was
renamed builtin blame), it was actually needed then, but I do see
what you are getting at.
It was no longer needed in commit 21666f1 ("convert object type
handling from a string to a number", 2007-02-26) with the removed
use of blob_type. I will update the commit message accordingly.
Thanks
On Fri, May 5, 2017 at 8:52 AM, Jeffrey Smith <whydoubt@xxxxxxxxx> wrote:
> While it was added there (it was part of builtin pickaxe which was
> renamed builtin blame), it was actually needed then, but I do see
> what you are getting at.
>
> It was no longer needed in commit 21666f1 ("convert object type
> handling from a string to a number", 2007-02-26) with the removed
> use of blob_type. I will update the commit message accordingly.
>
> Thanks
>
> P.S. Sorry for the duplicate message Ævar. I forgot to reply all
> the first time.
>
>
> On Fri, May 5, 2017 at 2:07 AM, Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>
> wrote:
>>
>> For commit message: This was originally added in commit acca687fa9
>> ("git-pickaxe: retire pickaxe", 2006-11-08), but has never been
>> needed.
>>
>> On Fri, May 5, 2017 at 7:27 AM, Jeff Smith <whydoubt@xxxxxxxxx> wrote:
>> > Signed-off-by: Jeff Smith <whydoubt@xxxxxxxxx>
>> > ---
>> > builtin/blame.c | 1 -
>> > 1 file changed, 1 deletion(-)
>> >
>> > diff --git a/builtin/blame.c b/builtin/blame.c
>> > index 07506a3..42c56eb 100644
>> > --- a/builtin/blame.c
>> > +++ b/builtin/blame.c
>> > @@ -8,7 +8,6 @@
>> > #include "cache.h"
>> > #include "refs.h"
>> > #include "builtin.h"
>> > -#include "blob.h"
>> > #include "commit.h"
>> > #include "tag.h"
>> > #include "tree-walk.h"
>> > --
>> > 2.9.3
>> >
>
>