On Wed, Feb 26, 2025 at 05:18:58PM +0100, Claudio Jeker wrote:
> While working on all the other bits in the session engine I came to the
> conclusion that if imsg_rde() fails the game is over. So just admit to
> that and fatal in that case.
> 
> This simplifies all the callers since there is no need to check the return
> value. This also allows some other functions to be come void.

ok

Should session_handle_{update,rrefresh}() also become void? Wouldn't
it be more consistent if the return (0) was pushed to parse_update()
and parse_rrefresh()?