<div dir="ltr"><br><br><div class="gmail_quote">2008/9/2 Olivier Goffart <span dir="ltr"><<a href="mailto:ogoffart@kde.org">ogoffart@kde.org</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Le lundi 1 septembre 2008, Alessandro Diaferia a écrit :<br>
<div class="Ih2E3d">> Ok, here comes the patch, even if it is almos the same as before =)<br>
> Please try this.<br>
<br>
</div>the bool KPasswordDialog::KPasswordDialogPrivate::isCapsLockToggled()<br>
will return an undefined value if Q_WS_X11 is not defined...  bad<br>
</blockquote><div><br>right! I added a return false outside the #ifdef =)<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>

Also, i think it may be safer to check if we have a keyEvent in the<br>
eventFilter.<br>
Otherwise there is a risk of infinite loop if ever the capsWarning function<br>
trigger an event.  (which is not impossible considered what it does)</blockquote><div> </div><div>Sure, fixed! <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<br>
--<br>
<font color="#888888">Olivier<br>
</font></blockquote></div><br>Unfortunately kdesu still doesn't make use of this eventFilter :(<br><br clear="all"><br>-- <br>Alessandro Diaferia<br>
</div>