<div dir="ltr"><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)"><br></div><div class="gmail_extra"><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">​Hello,<br><br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">New patches are here:<br><a href="http://patchwork.freedesktop.org/patch/51584">http://patchwork.freedesktop.org/patch/51584</a><br><a href="http://patchwork.freedesktop.org/patch/51585">http://patchwork.freedesktop.org/patch/51585</a>​</div><br><br><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">>​
What was the crash exactly?<br><br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">The crash​ happens because the seat is assigned an input method as soon<br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">as the text backend is initialized.<br><br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">So If you try to call text_input_activate() before text_backend_seat_created(),<br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">it will try to get an input_method that doesent exist from the seat.<br><br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">In theory this should never happen now because without a seat, text_input_activate<br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">should never be called. And if there is a seat it will have an input_method if<br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">the global is advertised(thanks to this patch).<br><br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">Is it better to check if seat->input_method is null just in case?<br><br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">to reproduce the crash you can just apply the second patch without the<br>first and use weston-editor to activate the input method.<br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)"><br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">Thanks<br></div><div class="gmail_default" style="font-family:monospace,monospace;font-size:small;color:rgb(68,68,68)">Murray.<br></div></div></div>