Re: [libgpiod][PATCH 0/4] dedicated examples

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jun 14, 2023 at 3:21 PM Kent Gibson <warthog618@xxxxxxxxx> wrote:
>
> On Wed, Jun 14, 2023 at 03:03:51PM +0200, Bartosz Golaszewski wrote:
> > >
> >
> > Great idea! I applied patches 1-3 with some tweaks (coding style, C++
> > comments, statify functions, put local functions into anonymous
> > namespaces for C++, apply black to python code etc. etc.).
> >
>
> Ok.  Would've been nice to get review comments before having changes
> pulled straight in, but whatever.
>

No functional changes. I wanted to avoid bikeshedding over coding style etc.

> Requiring C style comments in C++ is just weird.
>
> > You can respin the rust patch separately.
> >
>
> And what would you like respun there?
>

I saw comments from Erik and though you'd need a v2 but as it's mostly
a matter of taste, I'll apply it as is.

Bart

> Cheers,
> Kent.




[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux