Shade

surface=surf_Return;
SDL_FreeSurface(surf_Return);

Remember, both surface and surf_Return are pointer, and because you did an equal operator on them, they are both pointing to the same thing. So, when you call SDL_FreeSurface on one, it affects the other as well. Basically, your function should look like this:

SDL_Surface* CSurface::OnLoad(SDL_Surface *surface, TTF_Font *font, std::string str, SDL_Color clr){
if(surface) {
SDL_FreeSurface(surface);
}
if((surface=TTF_RenderText_Blended(font, str.c_str(), clr))==NULL){
return NULL;
}
return surface;
}

Also, I would probably call it PrintText or something, OnLoad doesn’t make much sense for what it’s doing.

SDL_Surface* CSurface::OnLoad(SDL_Surface *surface, TTF_Font *font, std::string str, SDL_Color clr){
SDL_Surface *surf_Return=NULL;
if((surf_Return=TTF_RenderText_Blended(font, str.c_str(), clr))==NULL){
return NULL;
}
surface=NULL;
surface=surf_Return;
SDL_FreeSurface(surf_Return);
return surface;
}

Thanks!
And I’ve tried so many things to get it to re-render but it simply doesn’t I’ve called both OnLoad() and OnDraw() in different places, e.g. in Point() when changing msg, I’ve also tried to call both in OnRender() but not even the initial text is shown now. I thought that the order of OnLoop() and OnRender() may play a roll in the main loop, but nope, nothing happend except for that bug that made me unable to close (or even minimize) the program
I don’t really understand what I’m doing wrong here…

In CApp.h:
#include
#include
using namespace std;
class…{
…
private:
SDL_Surface *surf_Text;
std::string msg;
TTF_Font *f;
};

in CApp::CApp(){
…
surf_Text=NULL;
msg=”Tic Tac Toe”;
}

in OnInit(){
if(TTF_Init()==-1){
return false;
}
if((f=TTF_OpenFont(“C:\\WINDOWS\\Fonts\\arial.ttf”, 36))==NULL){
return false;
}
SDL_Color color={255, 255, 255}; //for some reason I had to declare and initiate color simultaniously, it didn’t work to declare it in CApp.h and initiate it hear…
if((surf_Text=CSurface::OnLoad(f, msg, color))==NULL){
return false;
}
}
surf_Text is made with a function to load text. In CSurface.h:
#include
#include
…
public:
static SDL_Surface *OnLoad(TTF_Font *font, std::string str, SDL_Color clr);

In CSurface.cpp:
SDL_Surface* CSurface::OnLoad(TTF_Font *font, std::string str, SDL_Color clr){
SDL_Surface *surf_Return=NULL;
if((surf_Return=TTF_RenderText_Blended(font, str.c_str(), clr))==NULL){
return NULL;
}else
return surf_Return;
}

Now I should be able to use OnDraw() normally, so in OnRender(){
…
if(winner==X_WINS){
CSurface::OnDraw(surf_Display, surf_WinX, 100, 200);
CSurface::OnDraw(surf_Display, surf_Text, 200, 260);
}
…
SDL_Flip(surf_Display);
}

Also, in OnLoop.cpp I have a function stating:
void CApp::Points(int winner){
if(winner==1){
winsX++;
msg=”X: “+winsX++;
}else if(winner==2){
winsO++;
msg=”O: “+winsO++;
}
}

and lastly in OnCleanup(){
…
SDL_FreeSurface(surf_Text);
TTF_CloseFont(f);
TTF_Quit();
SDL_Quit();
}

I don’t get any errors but the text stays on “Tic Tac Toe”, which I think means it doesn’t update when msg is changed in Points(int);
(I also noticed that when I had this code implemented, I couldn’t close the application normally.)

Hopefully that was descriptive enough ^^’
Thanks again