[LTP] [PATCH 1/2] rtc01: add space to beginning of comments
Krzysztof Kozlowski
krzysztof.kozlowski@canonical.com
Fri Sep 10 14:46:39 CEST 2021
On 10/09/2021 11:14, Cyril Hrubis wrote:
> Hi!
>> Improve readability of comments - no functional change.
>
> Actually some of the comments are really commenting the obvious so it
> would be better just to remove most.
>
> For instance this:
>
> /* Read and alarm tests */
> read_alarm_test();
>
> How does the comment here add any value?
>
> And I would say that it's mostly the same for the ioclt() calls as well,
> the constants do have reasonable names so I guess that in cases like
> this:
>
> /* Read current alarm time */
> ret = ioctl(rtc_fd, RTC_ALM_READ, &rtc_tm);
>
> it's kind of obvious that we do read the alarm even without the comment.
Good point but also I am not sure if removing these comments is worth
patch on its own (without patch 2/2).
Best regards,
Krzysztof
More information about the ltp
mailing list