Re: [PATCH] log: properly handle decorations with chained tags
- From
brian m. carlson <sandals@crustytoothpaste.net>
- Date
- Dec 19, 2013, 03:18 UTC
- Message-ID
- <20131219031809.GG259467@vauxhall.crustytoothpaste.net>
- In-Reply-To
- <xmqq38lrknbd.fsf@gitster.dls.corp.google.com>
On Tue, Dec 17, 2013 at 04:36:06PM -0800, Junio C Hamano wrote:
> I think all we need to do, in addition to what the existing code > does, is to make sure that we _parse_ the object that the tag points > at, to avoid this problem. Something like this, perhaps, instead?
Yeah, that's the clean fix I was looking for, but couldn't quite come up with. I'm going to re-roll with your fix instead of mine and my tests. Any objections to adding your sign-off?
-- brian m. carlson / brian with sandals: Houston, Texas, US +1 832 623 2791 | http://www.crustytoothpaste.net/~bmc | My opinion only OpenPGP: RSA v4 4096b: 88AC E9B2 9196 305B A994 7552 F1BA 225C 0223 B187