Le 15/10/2013 18:30, Arnaud Thimel a écrit :
Bref, je suis pas pour du tout cette modification, mais j'imagine que tu l'as fait pour une bonne raison, quelle est-elle ?
Salut, En fait, sauf erreur de ma part, je n'ai remplacé que les corps des méthodes générées qui sont dépréciées. Cela permet de faciliter la migration (remplacer le code déprécié par le bon). En effet, il suffit au dév d'utiliser la fonction « inline method » de son IDE pour que l'IDE vire le code déprécié pour le remplacer par le bon. Or, les IDE (en tout cas, celui que j'utilise) sont pas très malin, si tu lui fais inliner une méthode avec une variable intermédiaire, il va t'ajouter une variable temporaire dans le code appelant. Par exemple, avec le code suivant List<Bateau> bateaux = dao.findAllByPort(port); Tu le inlines, il te génère List<Bateau> result1 = dao.forPortEquals(port).findAll(); List<Bateau> bateaux = result1; Avec ma modif, avec le inline, il génère ça : List<Bateau> bateaux = dao.forPortEquals(port).findAll(); Ce qui est ce qu'on attend. Voilà, j'ai donc fait cette modif pour faciliter le travail de migration pour le dév. Je suis d'accord que ça pose le problème que tu évoques (débug) mais normalement, le mec n'est pas censé appelé ce code vu qu'il est déprécié. Dans toutes les autres méthodes non dépréciées, j'ai bien laissé une variable intermédiaire comme d'habitude. -- Brendan Le Ny, Code Lutin bleny@codelutin.com (+33) 02 40 50 29 28