From mboxrd@z Thu Jan 1 00:00:00 1970 From: Karthik Nayak Subject: Re: [RFC/PATCH 07/11] branch: move 'current' check down to the presentation layer Date: Thu, 30 Jul 2015 00:14:14 +0530 Message-ID: References: <1438066594-5620-1-git-send-email-Karthik.188@gmail.com> <1438066594-5620-7-git-send-email-Karthik.188@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Matthieu Moy , Git , Christian Couder , Junio C Hamano To: Jacob Keller X-From: git-owner@vger.kernel.org Wed Jul 29 20:44:50 2015 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1ZKWLQ-0005nH-Vn for gcvg-git-2@plane.gmane.org; Wed, 29 Jul 2015 20:44:49 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751706AbbG2Soo (ORCPT ); Wed, 29 Jul 2015 14:44:44 -0400 Received: from mail-ob0-f179.google.com ([209.85.214.179]:34414 "EHLO mail-ob0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750839AbbG2Soo (ORCPT ); Wed, 29 Jul 2015 14:44:44 -0400 Received: by obre1 with SMTP id e1so14065152obr.1 for ; Wed, 29 Jul 2015 11:44:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=oZdkkLOR5mL5vaAjhdPklg6xy+aNBqJhGDqpjZJ4lMY=; b=ddkX528lhX99b8YX3CWDOd7wCmu8IHV2COhxl2VqAx7IPwcXV/2LBT3HE79jZdqWjb DWGKsG1o4crWNyneZyTjil2+Two/5T5E1RNHwgy9DqIZSPNYuvAFjY655DLrKi6TXCqU qqDwo/9Db1a0j3Kdd3ELjPwOsrDyWnVzzsi4LeEH/woN0hBavFjypN3BLfRa++341alM 643D9eMDa/g1RrgiZENpqmMISd5PP5Ug7KcnZhGtWjCnYSZwadSDke2Y5yjdOTk+LE9f JKWybs5kJ0aD3j0LlzFQVjOsycGmVfGNxEwlOAt6xxl+gaaSpNAc87xvWwaSgTM/8MUW gQ7A== X-Received: by 10.182.204.38 with SMTP id kv6mr41950553obc.70.1438195483375; Wed, 29 Jul 2015 11:44:43 -0700 (PDT) Received: by 10.182.26.73 with HTTP; Wed, 29 Jul 2015 11:44:14 -0700 (PDT) In-Reply-To: Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: On Wed, Jul 29, 2015 at 6:16 AM, Jacob Keller wrote: > On Tue, Jul 28, 2015 at 1:12 PM, Karthik Nayak wrote: >> On Tue, Jul 28, 2015 at 6:39 PM, Matthieu Moy >> wrote: >>> Karthik Nayak writes: >>> >>>> We check if given ref is the current branch in print_ref_list(). Move >>>> this check to print_ref_item() where it is checked right before >>>> printing. >>> >>> This means that the '*' and the different color are coded in C, hence >>> it's not possible to mimick this using "git for-each-ref --format ...". >>> >>> I do not consider this as blocking, but I think the ultimate goal should >>> be to allow this, so that all the goodies of "git branch" can be made >>> available to other ref-listing commands. >>> >> >> Not sure what you mean here. >> > > He means to make sure that any feature that was in tag, branch, > for-each-ref, etc should be available as part of ref-filter and in all > of them > > Maybe he misunderstood code or maybe you misunderstood the comment here? > > Regards, > Jake Ah, I didn't quite understand the first time. Thanks :) -- Regards, Karthik Nayak