[PATCH 11/20] rtems: Move _Partition_Is_buffer_on_boundary()
Sebastian Huber
sebastian.huber at embedded-brains.de
Fri Nov 20 16:38:33 UTC 2020
On 20/11/2020 17:10, Gedare Bloom wrote:
>> -/**
>> - * @brief Checks whether is on a valid buffer boundary for the_partition.
>> - *
>> - * This function returns TRUE if the_buffer is on a valid buffer
>> - * boundary for the_partition, and FALSE otherwise.
> I think keep these two lines. This function is not so clear without
> thinking through what it means to do pointer arithmetic (subtraction
> and mod).
>
>> - */
>> -RTEMS_INLINE_ROUTINE bool _Partition_Is_buffer_on_boundary (
>> - void *the_buffer,
>> - Partition_Control *the_partition
>> -)
The function name is _Partition_Is_buffer_on_boundary().
The brief description is "Checks whether is on a valid buffer boundary
for the_partition". This sentence is ill formed. If we disregard this,
the only extra information is "valid". So, maybe rename it to
_Partition_Is_buffer_on_valid_boundary().
The "This function returns TRUE if the_buffer is on a valid buffer
boundary for the_partition, and FALSE otherwise." tells you the obvious.
--
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/
More information about the devel
mailing list