ProjetM1MPRI2013 / central

Repo principal
6 stars 0 forks source link

Gestion mémoire dans le générateur de code #33

Open dbaelde opened 10 years ago

dbaelde commented 10 years ago

Bonjour @Nobody35. Si je lance le générateur de code avec valgrind, j'obtiens plusieurs occurrences de

==29255== Conditional jump or move depends on uninitialised value(s)
==29255==    at 0x40A806: Writer::writeType(Field*) (in /import/baelde/repo/mpri/generator/result)
==29255==    by 0x40C49E: Writer::writeHAction(PreClass*) (in /import/baelde/repo/mpri/generator/result)
==29255==    by 0x404842: Writer::write() (in /import/baelde/repo/mpri/generator/result)
==29255==    by 0x40EB96: Generator::write() (in /import/baelde/repo/mpri/generator/result)
==29255==    by 0x401D7E: main (in /import/baelde/repo/mpri/generator/result)

Je ne sais pas si c'est lié, mais Benoit a par ailleurs remarqué que la gestion mémoire était très manuelle dans le générateur de code: beaucoup de pointeurs, new, delete, alors qu'a priori il n'y a pas grand chose de dynamique là dedans.

Nobody35 commented 10 years ago

Je n'ai pas encore regardé valgrind sur le générateur. J'ai enlevé la plupart des pointeurs veux qui restent sont a gardé (il y a des partages de listes notamment) mais je n'ai pas encore ni justifié ni assuré qu'ils étaient bien delete(mais c'est prévu )

Pour les erreurs j'essaye de comprendre ce que ça signifie.

Rémy

Le 17 févr. 2014 à 09:56, David Baelde notifications@github.com a écrit :

Si je lance le générateur de code avec valgrind, j'obtiens plusieurs occurrences de

==29255== Conditional jump or move depends on uninitialised value(s) ==29255== at 0x40A806: Writer::writeType(Field) (in /import/baelde/repo/mpri/generator/result) ==29255== by 0x40C49E: Writer::writeHAction(PreClass) (in /import/baelde/repo/mpri/generator/result) ==29255== by 0x404842: Writer::write() (in /import/baelde/repo/mpri/generator/result) ==29255== by 0x40EB96: Generator::write() (in /import/baelde/repo/mpri/generator/result) ==29255== by 0x401D7E: main (in /import/baelde/repo/mpri/generator/result) Je ne sais pas si c'est lié, mais Benoit a par ailleurs remarqué que la gestion mémoire était très manuelle dans le générateur de code: beaucoup de pointeurs, new, delete, alors qu'a priori il n'y a pas grand chose de dynamique là dedans.

— Reply to this email directly or view it on GitHub.

Nobody35 commented 10 years ago

Bonjour J'ai laissé 4 pointeurs dans mon projet :

Les classes qui héritent de Toload (c'est expliqué dans le fichier) Le stack (mes fonctions marchent par effet de bord sur cet objet) le generateur (qui contient les loader et qui doit donc être passé ) les ifstream et ofstream (j'ai des problèmes de droit si je les passe en référence)

Rémy

Le 17 février 2014 14:34, Poulain poulainremy@gmail.com a écrit :

Je n'ai pas encore regardé valgrind sur le générateur. J'ai enlevé la plupart des pointeurs veux qui restent sont a gardé (il y a des partages de listes notamment) mais je n'ai pas encore ni justifié ni assuré qu'ils étaient bien delete(mais c'est prévu )

Pour les erreurs j'essaye de comprendre ce que ça signifie.

Rémy

Le 17 févr. 2014 à 09:56, David Baelde notifications@github.com a écrit :

Si je lance le générateur de code avec valgrind, j'obtiens plusieurs occurrences de

==29255== Conditional jump or move depends on uninitialised value(s) ==29255== at 0x40A806: Writer::writeType(Field) (in /import/baelde/repo/mpri/generator/result) ==29255== by 0x40C49E: Writer::writeHAction(PreClass) (in /import/baelde/repo/mpri/generator/result) ==29255== by 0x404842: Writer::write() (in /import/baelde/repo/mpri/generator/result) ==29255== by 0x40EB96: Generator::write() (in /import/baelde/repo/mpri/generator/result) ==29255== by 0x401D7E: main (in /import/baelde/repo/mpri/generator/result)

Je ne sais pas si c'est lié, mais Benoit a par ailleurs remarqué que la gestion mémoire était très manuelle dans le générateur de code: beaucoup de pointeurs, new, delete, alors qu'a priori il n'y a pas grand chose de dynamique là dedans.

Reply to this email directly or view it on GitHubhttps://github.com/ProjetM1MPRI2013/central/issues/33 .