Thank you for the clarification. I knew we discussed this recently and didn't recall the outcome. This is fine, we can be nicer than (say) compiler writers :)
On Wed, Jul 21, 2021 at 12:21 PM Sebastian Huber <sebastian.hu...@embedded-brains.de> wrote: > > On 21/07/2021 20:14, Gedare Bloom wrote: > >> + memset( attributes, 0, sizeof( *attributes ) ); > >> + > >> + if ( !bsp_interrupt_is_valid_vector( vector ) ) { > >> + return RTEMS_INVALID_ID; > >> + } > > I think do the error checking first, before changing the out parameters? > > Some users ignore return values. This way they get at least a > deterministic result from the directive. See: > > https://lists.rtems.org/pipermail/devel/2021-March/065840.html > > > -- > embedded brains GmbH > Herr Sebastian HUBER > Dornierstr. 4 > 82178 Puchheim > Germany > email: sebastian.hu...@embedded-brains.de > phone: +49-89-18 94 741 - 16 > fax: +49-89-18 94 741 - 08 > > 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@rtems.org http://lists.rtems.org/mailman/listinfo/devel