[PATCH 13/20] rtems: Move _Partition_Is_buffer_area_aligned()

Gedare Bloom gedare at rtems.org
Fri Nov 20 16:18:44 UTC 2020


On Fri, Nov 20, 2020 at 6:34 AM Sebastian Huber
<sebastian.huber at embedded-brains.de> wrote:
>
> On 20/11/2020 14:31, Joel Sherrill wrote:
>
> > Wasn't there Doxygen above it?
> No.
> >
> > Needs comments
>
> I think a one liner function with a very descriptive name needs no
> comments if it is placed in a source file:
>
> static bool _Partition_Is_buffer_area_aligned( const void
> *starting_address )
> {
>    return ( ( (uintptr_t) starting_address ) % CPU_SIZEOF_POINTER ) == 0;
> }

I disagree, this is worth one line to remind that buffers need to be
aligned to a multiple of the pointer size (and are a multiple of the
pointer size).

Just because we can put the code in one line doesn't mean that its
context is clear.

>
> --
> embedded brains GmbH
> Sebastian HUBER
> Dornierstr. 4
> 82178 Puchheim
> Germany
> email: sebastian.huber at embedded-brains.de
> Phone: +49-89-18 94 741 - 16
> Fax:   +49-89-18 94 741 - 08
> PGP: Public key available on request.
>
> embedded brains GmbH
> Registergericht: Amtsgericht München
> Registernummer: HRB 157899
> Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
> Unsere Datenschutzerklärung finden Sie hier: https://embedded-brains.de/datenschutzerklaerung/
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list