Re: [Lost] [Patch] Signale

[Toni Kaufmann <toni@xxxxxxxxxxxxxxxx>]

Kevin Wolf wrote:
> Toni Kaufmann schrieb:
>   
>> Hier ein Patch der LOST die wichtigsten Funktionen für Posix-Signale
>> beibringt. Dabei wird der bisherige signal-Syscall entfernt. Die  neue
>> implementierung funktioniert über send_message.
>>     
>
> Für Signale sind aber explizit die Funktionsnummern 256 bis 511 
> freigehalten. Ich halte es für eine schlechte Idee, die jetzt nicht zu 
> nutzen, sondern stattdessen einen nutzlosen Stringvergleich einzubauen 
> und die correlation_id zu mißbrauen.
>
> Für diesen teil daher ein klares NACK.
>
>   
Ok, ok
>> +/// LOST-Spezifisches Signal: Eingabedaten auf Terminal bereit
>> +#define SIGTERMINPUT 31
>>     
>
> Wollen wir Signale wirklich aktiv nutzen und nicht nur für die 
> POSIX-Kompatibilität halbwegs abbilden? Für LOST-interne Sachen würde 
> ich einen normalen RPC bevorzugen (wobei mir ohnehin nicht klar ist, 
> wofür dieses Signal gut ist)
>   
Da frage ich mich ehrlich gesagt auch, was ich da machen wollte... da
habe ich wohl mal was angefangen, und nich wieder weggeputzt ;-)

>> +/**
>> + * Handler fuer ein ignoriertes Signal
>> + *
>> + * @param signum Signalnummer
>> + */
>> +void _signnal_ignore_handler(int signum)
>> +{
>> +}
>>     
>
> Unnötig, wofür gibt es NULL?
>   
Da war zuerst mal Null... aber dann musste ich SIG_ERR irgendwie noch
definieren. Und ich glaube es ist nicht gerade geschickt, wenn SIG_ERR
== SIG_IGN