> Ah, I didn't make the non-nit part of that review clear:
>
> Non-nit:
> mir_demo_client_display_config crashes if you press ‘p’ before making any
> display configuration changes, due to print_current_configuration() not having
> a null-check on conf and nothing else assigns to conf in the startup process.
No, that was clear. I failed to push the correction.
> Ah, I didn't make the non-nit part of that review clear: client_ display_ config crashes if you press ‘p’ before making any configuration( ) not having
>
> Non-nit:
> mir_demo_
> display configuration changes, due to print_current_
> a null-check on conf and nothing else assigns to conf in the startup process.
No, that was clear. I failed to push the correction.