Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dead locks #54

Open
FastFrench opened this issue Mar 1, 2013 · 9 comments
Open

Dead locks #54

FastFrench opened this issue Mar 1, 2013 · 9 comments
Labels

Comments

@FastFrench
Copy link
Member

Hello,

juste pour info, il y a quelques problèmes de lock. Les plus sérieux étant dans ServerConnection, où censuré.

De plus, il est fortement déconseillé d'utiliser lock(this), surtout quand ce sont des objets publics.

Merci,

FF

@Timorem
Copy link
Member

Timorem commented Mar 1, 2013

Tu es sûr qu'il y a deadlock, les appels viennent du même thread non ?

@FastFrench
Copy link
Member Author

Effectivement, le lock ne bloque pas le thread qui a posé le lock.

En tous cas, j'arrive régulièrement sur des blocages sur des lock (this), donc je suis sûr du constat. Cependant tu as raison, les deux exemples que j'ai donné ne sont pas concernés.

J'ai pas mal limité le problème en reduisant les zones lockées le plus possible.

@Timorem
Copy link
Member

Timorem commented Mar 1, 2013

Dans ton exemple tu n'es pas bloqué dans Bar puisque l'appel vient du même thread.
Mais oui faut que je revois ça

@FastFrench
Copy link
Member Author

Quel Bar ? sifflote Une petite soif ?

@ghost
Copy link

ghost commented Mar 1, 2013

Il a de l'humour en plus.

@FastFrench
Copy link
Member Author

Bah, il a oublié de poser un lock sur mon post quand il a commencé à rédiger sa réponse.

@vendethiel
Copy link
Contributor

pas moyen de lock un message sur gh, hein

@Timorem
Copy link
Member

Timorem commented Mar 1, 2013

C'est toi la lock

@vendethiel
Copy link
Contributor

idd

indeed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants