Re: [PATCH]: uniq: add "--group" option

[Pádraig Brady]

On 02/21/2013 07:40 PM, Assaf Gordon wrote:
> Assaf Gordon wrote, On 02/21/2013 11:37 AM:
>>
>> You were planning on "--group" to mean explicitly "output all input lines, 
>> and add group-markers for unique groups" (meaning -u/-d/-D and --group are 
>> mutually exclusive).
>>
> 
> Attached is a version that behaves as previously discussed.
> "--group" can't be used with -c/-d/-D/-u.
> 
> Since it's a completely separate behavior, I found it easier to create a 
> whole new code path in "check_file()" for the special case of grouping.
> 
> Comments are welcomed,
>  -gordon

I'll push this tomorrow with the attached changes.
I added NEWS, docs and refactored the
default and --group core loops together as
as they're essentially the same.

thanks!
Pádraig.

uniq--group--adjustments.diff
Description: Text Data