lkml.org 
[lkml]   [2022]   [Mar]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v1 4/4] auxdisplay: lcd2s: Use array size explicitly in lcd2s_gotoxy()
    On Sat, Mar 19, 2022 at 6:46 AM Miguel Ojeda
    <miguel.ojeda.sandonis@gmail.com> wrote:
    > On Tue, Mar 8, 2022 at 4:11 PM Andy Shevchenko
    > <andriy.shevchenko@linux.intel.com> wrote:
    > >
    > > Currently the reading of the onstack array is confusing since two
    > > out of three members are of different types. Let it be more clear
    > > by explicitly set the array size, so everybody will understand that
    > > parameters are cast to the type of the array.
    >
    > I am not sure what you mean by this, could you please elaborate why is
    > it more clear if the size is given?

    With [] and parameters not being bytes by type one needs an additional
    processing step to figure that out. When I see u8 ...[3] I immediately
    understand that there are 3 _bytes_. Moreover, [] is error prone when
    one may add something to the buffer with the expectation that it will
    work somehow (for example, with an old device you need 3, with new
    device revision you need 4 bytes and passing 4 bytes breaks the old
    device). Of course most of that quite likely won't happen.

    If you think this change is not worth it, please, drop.


    --
    With Best Regards,
    Andy Shevchenko

    \
     
     \ /
      Last update: 2022-03-19 12:38    [W:3.763 / U:0.200 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site