Liberation de la memoire en Multi-Thread

ou mieux encore :

Code:

---

1 2 3 4 5 6 7 8

if (fileName != "NOP") { cout << "File received, current processing file !" << endl; pData = (PMYDATA) HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(MYDATA)); pData->fileNameLink = strdup(fileName.c_str()); hThread = CreateThread(NULL, 0, lectureFichierGPS, pData, 0, NULL); CloseHandle(hThread); }

---

Ca evite d'avoir le fileNameChar

Effectivement, c'est mieux ! :D
Ca donne quoi alors maintenant?

Toujours des fuites de mémoires mais le code me plait dejà beaucoup plus...
Je pense que j'ai des fonctions dans mon code qui merdouille....

Il faut que tu regardes à chaque fois que tu utilises à pointeur.
A part ça, je sais pas trop quoi te dire :roll:

Pfiou...faut que je reprenne tout mon code moi... aller je m'y met, je te tiens au courant et merci encore pour ton aide!

ha si une question (ou deux) :
Quand on fait :

Code:

---

1 2 3

FILE *file; file = fopen(fileName.c_str(), "rb"); fclose(file);

---

faut il faire appres :

Code:

---

delete(file);

---

?

Là je n'ai pas la réponse avec certitude, mais à priori je dirais que non, tu n'as pas besoin de faire le delete et que le fclose fait tout ce qu'il faut.
A vérifier :mrgreen:

Non.

De plus, il ne faut pas faire le fclose() si fopen() a renvoyé NULL:
free(NULL) est garanti ne rien faire, mais fclose(NULL) est un comportement indéfini (et pour cette raison, plante exprès sous Visual 2005).

et dans un cas comme celui-ci :

Code:

---

1 2 3 4

{ char *maxSoftsTmp = c_inifile_get_string("SOFTS", "TOTAL", NULL); maxSofts = strtol(maxSoftsTmp, NULL, 0); }

---

C'est marqué dans la doc :roll:

OUPS!
:sm: ça t'apprendra verbal à pas lire les docs!

J'ai ça :

Code:

---

1 2 3 4 5 6 7

while (x <= maxSofts) { char ch[3]; sprintf(ch, "%d", x); listSofts[x-1] = c_inifile_get_string("SOFTS", ch, NULL); x++; }

---

J'ai voulu faire un

Code:

---

free(listSofts);

---

mais ça plante
Je pense devoir utiliser :

Code:

---

delete(listSofts[]);

---

Mais la difference avec free() ?


edito :
pour eviter le char ch[3] si je mets ça ça va? :

Code:

---

1 2 3 4 5

while (x <= maxSofts) { listSofts[x-1] = c_inifile_get_string("SOFTS", (const char*) x, NULL); x++; }

---

mais ça plante avec ça...

le char ch[3], aucun problème, il est sur la pile et n'est pas utilisé après le while.

Par contre:

• On ne sait pas comment est alloué listSofts
• Il faut supprimer les éléments de listSofts comme suit:
  
  Code:

  ---

  1 2 3 4

  for(int iSoft=0 ; iSoft<maxSofts ; iSoft++) { free(listSofts[iSoft]), listSofts[iSoft]=NULL; }

  ---

Allocation :

Code:

---

1 2 3 4 5 6 7 8

string listSofts[maxSofts]; while (x <= maxSofts) { char ch[3]; sprintf(ch, "%d", x); listSofts[x-1] = c_inifile_get_string("SOFTS", ch, NULL); x++; }

---

Alors là, problème: Le char* retourné par c_inifile_get_string est perdu dans la boucle même.
Change-là ainsi:

Code:

---

1 2 3 4 5 6 7 8 9

string listSofts[maxSofts]; for(x=1 ; x<=maxSofts ; x++) { char ch[3]; sprintf(ch, "%d", x); char * iniStr = c_inifile_get_string("SOFTS", ch, NULL); listSofts[x-1] = iniStr; free(iniStr); }

---

Et tu n'auras rien à faire pour supprimer, puisque le tableau est sur la pile.

Merci encore médinoc, peux tu m'expliquer un tout petit plus (ouai j'aime bien quand ça marche mais je prefere encore mieux comprendre).
Tu me dis que le tableau est dans la "pile"... mais le tableau est initialisé avant le { par string ...

Ben c'est un tableau de maxSoft objets std::string, tous dans la pile.
Typiquement, chaque objet string doit contenir un pointeur vers ses données, elles allouées sur le tas...

Ok merci pour vos explications ça m'aide vraiment.
Par contre j'ai toujours des fuites moi (je parle de mémoire là hein...:?)

Jette un oeil du côté de DEBUG_NEW pour t'aider à savoir d'où tes fuites (:mrgreen:) viennent.

merci pour l'info mais je n'arrive pas à l'utiliser, je travail sous code::blocks, ming pour la compilation et gdb comme debugger (que je ne sais vraiment pas utiliser...)