Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-09 Thread Karthik Nayak
On Sun, Aug 9, 2015 at 1:49 PM, Eric Sunshine wrote: > On Sun, Aug 9, 2015 at 4:09 AM, Karthik Nayak wrote: >> On Sun, Aug 9, 2015 at 1:34 PM, Eric Sunshine >> wrote: >>> On Sun, Aug 9, 2015 at 2:55 AM, Karthik Nayak wrote: On Sun, Aug 9, 2015 at 9:12 AM, Eric Sunshine wrote: >

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-09 Thread Eric Sunshine
On Sun, Aug 9, 2015 at 4:09 AM, Karthik Nayak wrote: > On Sun, Aug 9, 2015 at 1:34 PM, Eric Sunshine wrote: >> On Sun, Aug 9, 2015 at 2:55 AM, Karthik Nayak wrote: >>> On Sun, Aug 9, 2015 at 9:12 AM, Eric Sunshine >>> wrote: That raises another question. Why are 'struct ref_formatting_sta

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-09 Thread Karthik Nayak
On Sun, Aug 9, 2015 at 1:30 PM, Matthieu Moy wrote: > > > Le 8 août 2015 09:03:03 GMT+02:00, Karthik Nayak a > écrit : >>On Fri, Aug 7, 2015 at 9:34 AM, Eric Sunshine >>wrote: >>> On Wed, Aug 5, 2015 at 2:54 PM, Karthik Nayak >>wrote: Implement an `align` atom which will act as a modifier

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-09 Thread Karthik Nayak
On Sun, Aug 9, 2015 at 1:34 PM, Eric Sunshine wrote: > On Sun, Aug 9, 2015 at 2:55 AM, Karthik Nayak wrote: >> On Sun, Aug 9, 2015 at 9:12 AM, Eric Sunshine >> wrote: >>> On Sat, Aug 8, 2015 at 2:35 AM, Karthik Nayak wrote: On Fri, Aug 7, 2015 at 8:57 AM, Eric Sunshine wrote: >

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-09 Thread Eric Sunshine
On Sun, Aug 9, 2015 at 2:55 AM, Karthik Nayak wrote: > On Sun, Aug 9, 2015 at 9:12 AM, Eric Sunshine wrote: >> On Sat, Aug 8, 2015 at 2:35 AM, Karthik Nayak wrote: >>> On Fri, Aug 7, 2015 at 8:57 AM, Eric Sunshine >>> wrote: On Wed, Aug 5, 2015 at 2:54 PM, Karthik Nayak wrote:

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-09 Thread Matthieu Moy
Le 8 août 2015 09:03:03 GMT+02:00, Karthik Nayak a écrit : >On Fri, Aug 7, 2015 at 9:34 AM, Eric Sunshine >wrote: >> On Wed, Aug 5, 2015 at 2:54 PM, Karthik Nayak >wrote: >>> Implement an `align` atom which will act as a modifier atom and >align >>> any string with or without an %(atom) appea

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-08 Thread Karthik Nayak
On Sun, Aug 9, 2015 at 9:12 AM, Eric Sunshine wrote: > On Sat, Aug 8, 2015 at 2:35 AM, Karthik Nayak wrote: >> On Fri, Aug 7, 2015 at 8:57 AM, Eric Sunshine >> wrote: >>> On Wed, Aug 5, 2015 at 2:54 PM, Karthik Nayak wrote: of the padding to be performed. If the atom length is more than t

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-08 Thread Eric Sunshine
On Sat, Aug 8, 2015 at 2:35 AM, Karthik Nayak wrote: > On Fri, Aug 7, 2015 at 8:57 AM, Eric Sunshine wrote: >> On Wed, Aug 5, 2015 at 2:54 PM, Karthik Nayak wrote: >>> of the padding to be performed. If the atom length is more than the >>> padding length then no padding is performed. e.g. to pad

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-08 Thread Karthik Nayak
On Fri, Aug 7, 2015 at 9:34 AM, Eric Sunshine wrote: > On Wed, Aug 5, 2015 at 2:54 PM, Karthik Nayak wrote: >> Implement an `align` atom which will act as a modifier atom and align >> any string with or without an %(atom) appearing before a %(end) atom >> to the right, left or middle. >> >> It is

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-07 Thread Karthik Nayak
On Fri, Aug 7, 2015 at 8:57 AM, Eric Sunshine wrote: > On Wed, Aug 5, 2015 at 2:54 PM, Karthik Nayak wrote: >> Implement an `align` atom which will act as a modifier atom and align >> any string with or without an %(atom) appearing before a %(end) atom >> to the right, left or middle. > > For som

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-06 Thread Eric Sunshine
On Wed, Aug 5, 2015 at 2:54 PM, Karthik Nayak wrote: > Implement an `align` atom which will act as a modifier atom and align > any string with or without an %(atom) appearing before a %(end) atom > to the right, left or middle. > > It is followed by `:,`, where the `` is > either left, right or mi

Re: [PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-06 Thread Eric Sunshine
On Wed, Aug 5, 2015 at 2:54 PM, Karthik Nayak wrote: > Implement an `align` atom which will act as a modifier atom and align > any string with or without an %(atom) appearing before a %(end) atom > to the right, left or middle. For someone not familiar with the evolution of this patch series, "al

[PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-05 Thread Karthik Nayak
Implement an `align` atom which will act as a modifier atom and align any string with or without an %(atom) appearing before a %(end) atom to the right, left or middle. It is followed by `:,`, where the `` is either left, right or middle and `` is the total length of the padding to be performed. I

[PATCH v9 03/11] ref-filter: implement an `align` atom

2015-08-04 Thread Karthik Nayak
Implement an `align` atom which will act as a modifier atom and align any string with or without an %(atom) appearing before a %(end) atom to the right, left or middle. It is followed by `:,`, where the `` is either left, right or middle and `` is the total length of the padding to be performed. I