[PATCH v2] Improve support for long options to grep

[Walter Lozano]

On 1/26/22 16:12, Walter Lozano wrote:
> 
> 
> On 1/26/22 15:38, Xabier Oneca -- xOneca wrote:
>> Hi Walter,
>>
>>      > There is also unnecessary duplication of source code.
>>      >
>>      > Just use a #define and C string concatenation to add the extra
>>      > long option strings.
>>
>>     Yes, I totally agree you. However I used this approach since a 
>> previous
>>     patch I submitted was rewritten in this way [1] when applying to 
>> master.
>>
>>
>> Maybe a better approach is to always use getopt32long, as it is 
>> converted to getopt32 if LONG_OPTS is not enabled. Maybe Denys forgot 
>> about this.
>> See libbb.h: 
>> https://git.busybox.net/busybox/tree/include/libbb.h#n1351 
>> <https://git.busybox.net/busybox/tree/include/libbb.h#n1351>
> 
> Thank you for the pointer. Yes, it looks much cleaner to always use 
> getopt32long, not sure if there was some reason I'm not aware to try to 
> avoid it.
> 

Denys any preference in how to handle this to move things forwards?

Regards,

-- 
Walter Lozano
Collabora Ltd.