Re: [PATCH v4] gitweb: ref markers link to named shortlogs
- From
Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
- Date
- Aug 22, 2008, 13:20 UTC
- Message-ID
- <cb7bb73a0808220620m10245693pabe49993e91cff3d@mail.gmail.com>
- In-Reply-To
- <200808221501.54908.jnareb@gmail.com>
On Fri, Aug 22, 2008 at 3:01 PM, Jakub Narebski <jnareb@gmail.com> wrote:
>
> Seems overly complicated. How about something like this, instead?
> It simply moves stripping ^{} from refs to format_ref_marker(),
> and uses "tags/v1.5.0^{}" instead of [ "tags/v1.5.0", 1 ].My thought was that if the refs stuff was being used for something else than format_ref_marker, it would fail more spectacularly due to the different kind of values, rather than introducing potentially subtle bugs due to the additiona ^{} popping in unexpectedly, but it wouldn't be a problem to do it the simple way. Let me clean the patch up and resubmit it.
-- Giuseppe "Oblomov" Bilotta