Rockbox mail archive

• Next message: Linus Nielsen Feltzing: "Re: cvs: apps plugin.c,1.18,1.19"
• Previous message: Linus Nielsen Feltzing: "Re: Click when Pressing Next"
• Next in thread: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
• Reply: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ] [ attachment ]

J?rg Hohensohn wrote:
> +/*---------------------------------------------------------------------------
> + * Remove a thread from the scheduler.
> + * Parameter is the ID as returned from create_thread().
> + *---------------------------------------------------------------------------
> + */
> +void remove_thread(int threadnum)

Jörg,

I think you need to adjust current_thread as well, so it

a) isn't out of range

b) is pointing to the currently running thread

Also, I don't see you handling the case when a thread is deleting itself.

Linus
_______________________________________________
http://cool.haxx.se/mailman/listinfo/rockbox

• Next message: Linus Nielsen Feltzing: "Re: cvs: apps plugin.c,1.18,1.19"
• Previous message: Linus Nielsen Feltzing: "Re: Click when Pressing Next"
• Next in thread: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
• Reply: [IDC]Dragon: "Re: cvs: firmware thread.c,1.27,1.28"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ] [ attachment ]