On 3/18/2024 10:12 AM, Ani Sinha wrote:
>
>
<snip>
>>> + }
>>> +
>>> + if (strlen(output_str)) {
>>> + output_str[strlen(output_str) - 1] = '\0';
>>
>> You don't need this since you're using strncat which adds its own '\0'.
>
> If I understand this correctly, this code simply eliminates the extra “,”
> character in the end. Therefore it is needed.
> Since it is not obvious, in the previous review and before, I asked the
> author to add a comment to explain this clearly.
>
>> I wasn't quite able to follow along
>> on the discussion between Ani and you, so putting this in here in case it
>> wasn't already mentioned.
>>
Ah, great, that makes sense. I did see that it was destroying data but didn't
spend enough time to think through
what data it was destroying, and if that was a feature or a bug. Thanks for
calling it out!
- Easwar
