SFML community forums
Help => General => Topic started by: Beta_Ravener on October 20, 2011, 07:58:35 pm
-
My 2 programs using SFML 2.0 have both some strange glitch, that doesn't affect them but it's annoying. I'm creating sf::Window and sf::Render window in first and sf::Window in second and both programs are using also console window from sfml-main. When those windows appers on screen they behave correct and everything, but closing them through X in right up corner cause the created windows disappear (I think they close correctly), but that console window stays stuck for 3-4 seconds and then it crash. Note that when I close window my programs end(because of loop that uses sf::Window::IsOpened()). I get access violation, but it's not in my code and gets thrown after my main() function returns and formally ends program. The strange thing is, that when program is closed by clicking X in corner of console window everything closes correctly and no error appears.
I'm still not sure if it's bug in my code or SFML, because I was unable to jump from my code to SFML files. It can be still that I forgot to do something with resource inside my code, but I checked it few times and made destructors for almost every class cleaning those resources before program ends. Didn't help though, so I deleted those destructors for not to make more mess than there already is.
And sorry if there's topic like this, I couldn't find anything helpful (other than some topic about SFML using globals bacause of some ATI driver bug.. so maybe it'd help to say that I'm using ATI).
-
Ok I got something.. though I still can't jump to sfml source to track it down, I just noticed that upon exit in debug mode I got this line in console: "An internal OpenGL call failed in Image.cpp (90) : GL_INVALID_OPERATION, the specified operation is not allowed in the current state".
Also in debug mode, visual studio can look into disassmbly and tell me some info. Well when my main function returns, it's Main thread[5092] and stack frame is Waves.exe!_tmainCRTStartup(and some more info), but it stays here for about 6 steps. Then it executes: "0141A10F call dword ptr [__imp__exit (1430B04h)] ". It gets through msvcr90d.dll, kernel32.dll, ntdll.dll, and maybe others.. it's probably cleaning this debug info. But what's interesting, it finally gets to that line causing troubles: "69092994 mov edi,dword ptr [eax+8] " in Main Thread[5092] and stack frame is "atioglxx.dll!69092994()" .
Hope this helps somehow
-
I'm still not sure if it's bug in my code or SFML
care to show your code?
-
care to show your code?
Sure.. in fact while trying to get together some code to post here, I've found exact place where this happens. It happens when I call single function in my first program with 2 windows (the second one is out because it had the very same error but it was because of one vector).
This is the code for rendering second window, which is something like child to first, because it closes when the first one closes.
void SettingsUI::Render(){
window.SetActive();
window.Clear(sf::Color(0,0,0));
sf::Text text;
text.SetCharacterSize(20);
text.SetColor(sf::Color(30, 255, 30));
unsigned int i;
for(i = 0; i < menuSize; i++){
text.SetString(menuText[i]);
text.SetPosition(10.f, i*25.f+10.f);
window.Draw(text);
}
text.SetString("Input: " + input);
text.SetPosition(10.f, i*25.f+30.f);
window.Draw(text);
window.Display();
window.SetActive(false);
}
If I won't call this from my main function, the program ends without error. Playing with those commands I found that the only line causing trouble is: window.Draw(text); If you comment both of them away, text is not displayed but no error occurs on closing.
It may help to say that I initialize window this way:
window.Create(sf::VideoMode(500, 300, 32), "Settings", sf::Style::Titlebar);
and close like this:
void SettingsUI::Close(){
window.Close();
menuText.clear();
waveEngine = 0;
}
Also note that std::vector<sf::String> menuText;
sf::String input;
-
It's probably the known bug of the default font which causes a crash at global exit.
-
Then it should be true, that default font is loaded only if no other font is bind to sf::Text while calling sf::RenderWindow::Draw ? If so, would loading my own font to sf::Font and using this one prevent the crash?
And why closing console window doesn't result it crash? The console window, however, stays stuck for some time, and at last the programs ends returning exit code -1073741510 (but at the end of main I return 0).
-
Then it should be true, that default font is loaded only if no other font is bind to sf::Text while calling sf::RenderWindow::Draw ? If so, would loading my own font to sf::Font and using this one prevent the crash?
To prevent the crash, the default font must never be instanciated, so you have to pass your own font to the constructor of all your sf::Text instances.
Doing it with the SetFont function is already too late.
And why closing console window doesn't result it crash?
Closing the console terminates the program in a very ugly way -- nothing gets destructed properly. So no "crash", but memory leaks etc.
-
Closing the console terminates the program in a very ugly way -- nothing gets destructed properly. So no "crash", but memory leaks etc.
That's quite disturbing as I'm planning to use console in final release.. when we are at it, is there any way to disable all the options to close console?
-
That's quite disturbing as I'm planning to use console in final release..
That's fine, just don't close it directly.
when we are at it, is there any way to disable all the options to close console?
I don't think that Windows allows this.
-
Thanks for the help.
And one more thing, though this might be already known. I didn't want to have to include font with my program and as the built-in arial was pretty good, I just did:
sf::Font font = sf::Font::GetDefaultFont();
and this one I passed into sf::Text constructor. The result is same as before, but no crash anymore. As I read in that other thread, you got problems with those global variables, that can't be initialized before entering main function. Maybe you could have a class inside SFML that groups all the globals and something like sf::Init() that would user call from body of his main? It's the very same as glewInit() I need to call before I can use many of openGL calls, and there probably isn't better way to do this.
-
The result is same as before, but no crash anymore.
That's... strange :?
As I read in that other thread, you got problems with those global variables, that can't be initialized before entering main function. Maybe you could have a class inside SFML that groups all the globals and something like sf::Init() that would user call from body of his main? It's the very same as glewInit() I need to call before I can use many of openGL calls, and there probably isn't better way to do this.
It's a little bit more complicated ;)
-
Laurent
I really need a fix for this text bug very soon. I can guarantee that I don't need the default font, so can I comment out some code in the libraries to stop the default font being instanciated? If so, can you tell me which lines?
I don't care what I have to do to fix this, but I need to do it now!
-
You can comment every call to Font::GetDefaultFont (should occur in sf::Text constructors only), and assign NULL instead.
-
So what should the Text.hpp say?
explicit Text(const String& string, const Font& font = Font::GetDefaultFont(), unsigned int characterSize = 30);
I assume it needs to call a function to just return a NULL sf::Font ??
-
Just remove the default argument. If you really need to construct sf::Text instances from just a string, add a new constructor:
Text(const String& string);
-
Annoyingly, that didn't fix it.
This is my stack trace, if it's any help.
> PR1037.exe!sf::priv::MutexImpl::Lock() Line 52 + 0xc bytes C++
PR1037.exe!sf::Mutex::Lock() Line 62 C++
PR1037.exe!sf::Lock::Lock(sf::Mutex & mutex) Line 39 C++
PR1037.exe!sf::priv::WglContext::CreateContext(sf::priv::WglContext * shared, unsigned int bitsPerPixel, const sf::ContextSettings & settings) Line 315 + 0x10 bytes C++
PR1037.exe!sf::priv::WglContext::WglContext(sf::priv::WglContext * shared) Line 59 C++
PR1037.exe!sf::priv::GlContext::New() Line 145 + 0x28 bytes C++
PR1037.exe!`anonymous namespace'::GetInternalContext() Line 90 + 0x5 bytes C++
PR1037.exe!sf::priv::GlContext::EnsureContext() Line 138 + 0x7 bytes C++
PR1037.exe!sf::GlResource::EnsureGlContext() Line 86 C++
PR1037.exe!sf::Texture::~Texture() Line 77 C++
PR1037.exe!sf::Font::Page::~Page() + 0x4e bytes C++
-
Can I see the full stack trace?
-
Sure!
ntdll.dll!777518d7()
[Frames below may be incorrect and/or missing, no symbols loaded for ntdll.dll]
PR1037.exe!sf::priv::MutexImpl::Lock() Line 52 + 0xc bytes C++
PR1037.exe!sf::Mutex::Lock() Line 62 C++
PR1037.exe!sf::Lock::Lock(sf::Mutex & mutex) Line 39 C++
> PR1037.exe!sf::priv::WglContext::CreateContext(sf::priv::WglContext * shared, unsigned int bitsPerPixel, const sf::ContextSettings & settings) Line 315 + 0x10 bytes C++
PR1037.exe!sf::priv::WglContext::WglContext(sf::priv::WglContext * shared) Line 59 C++
PR1037.exe!sf::priv::GlContext::New() Line 145 + 0x28 bytes C++
PR1037.exe!`anonymous namespace'::GetInternalContext() Line 90 + 0x5 bytes C++
PR1037.exe!sf::priv::GlContext::EnsureContext() Line 138 + 0x7 bytes C++
PR1037.exe!sf::GlResource::EnsureGlContext() Line 86 C++
PR1037.exe!sf::Texture::~Texture() Line 77 C++
PR1037.exe!sf::Font::Page::~Page() + 0x4e bytes C++
PR1037.exe!std::_Pair_base<unsigned int const ,sf::Font::Page>::~_Pair_base<unsigned int const ,sf::Font::Page>() + 0x19 bytes C++
PR1037.exe!std::pair<unsigned int const ,sf::Font::Page>::~pair<unsigned int const ,sf::Font::Page>() + 0x16 bytes C++
PR1037.exe!std::pair<unsigned int const ,sf::Font::Page>::`scalar deleting destructor'() + 0x16 bytes C++
PR1037.exe!std::_Destroy<std::pair<unsigned int const ,sf::Font::Page> >(std::pair<unsigned int const ,sf::Font::Page> * _Ptr) Line 64 C++
PR1037.exe!std::allocator<std::pair<unsigned int const ,sf::Font::Page> >::destroy(std::pair<unsigned int const ,sf::Font::Page> * _Ptr) Line 213 + 0x9 bytes C++
PR1037.exe!std::_Dest_val<std::allocator<std::pair<unsigned int const ,sf::Font::Page> >,std::pair<unsigned int const ,sf::Font::Page> >(std::allocator<std::pair<unsigned int const ,sf::Font::Page> > & _Alval, std::pair<unsigned int const ,sf::Font::Page> * _Pdest) Line 288 C++
PR1037.exe!std::_Tree<std::_Tmap_traits<unsigned int,sf::Font::Page,std::less<unsigned int>,std::allocator<std::pair<unsigned int const ,sf::Font::Page> >,0> >::_Erase(std::_Tree_nod<std::_Tmap_traits<unsigned int,sf::Font::Page,std::less<unsigned int>,std::allocator<std::pair<unsigned int const ,sf::Font::Page> >,0> >::_Node * _Rootnode) Line 1617 + 0x22 bytes C++
PR1037.exe!std::_Tree<std::_Tmap_traits<unsigned int,sf::Font::Page,std::less<unsigned int>,std::allocator<std::pair<unsigned int const ,sf::Font::Page> >,0> >::clear() Line 1416 C++
PR1037.exe!sf::Font::Cleanup() Line 390 C++
PR1037.exe!sf::Font::~Font() Line 100 C++
PR1037.exe!TextElement::~TextElement() Line 60 + 0x66 bytes C++
PR1037.exe!std::_Pair_base<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement>::~_Pair_base<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement>() + 0x54 bytes C++
PR1037.exe!std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement>::~pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement>() + 0x2b bytes C++
PR1037.exe!std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement>::`scalar deleting destructor'() + 0x2b bytes C++
PR1037.exe!std::_Destroy<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >(std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> * _Ptr) Line 64 C++
PR1037.exe!std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >::destroy(std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> * _Ptr) Line 213 + 0x9 bytes C++
PR1037.exe!std::_Dest_val<std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >(std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> > & _Alval, std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> * _Pdest) Line 288 C++
PR1037.exe!std::_Tree<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> >::_Erase(std::_Tree_nod<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> >::_Node * _Rootnode) Line 1617 + 0x22 bytes C++
PR1037.exe!std::_Tree<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> >::_Erase(std::_Tree_nod<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> >::_Node * _Rootnode) Line 1615 C++
PR1037.exe!std::_Tree<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> >::clear() Line 1416 C++
PR1037.exe!std::_Tree<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> >::erase(std::_Tree_const_iterator<std::_Tree_val<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> > > _First, std::_Tree_const_iterator<std::_Tree_val<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> > > _Last) Line 1383 C++
PR1037.exe!std::_Tree<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> >::_Tidy() Line 1866 + 0xaa bytes C++
PR1037.exe!std::_Tree<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> >::~_Tree<std::_Tmap_traits<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> >,0> >() Line 792 C++
PR1037.exe!std::map<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> > >::~map<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,TextElement,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > >,std::allocator<std::pair<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const ,TextElement> > >() + 0x2b bytes C++
PR1037.exe!`dynamic atexit destructor for 'TextManager::textData''() + 0x28 bytes C++
PR1037.exe!doexit(int code, int quick, int retcaller) Line 567 C
PR1037.exe!exit(int code) Line 393 + 0xd bytes C
PR1037.exe!__tmainCRTStartup() Line 284 C
PR1037.exe!WinMainCRTStartup() Line 189 C
kernel32.dll!7635339a()
ntdll.dll!77729ed2()
ntdll.dll!77729ea5()
-
This one is your fault: you have a global sf::Font instance. This messes up with SFML's globals, and produces the same bug as the default font.
Have you tried to link SFML statically?
-
mutex contains:
- mutex {myMutexImpl=0x0051b3d8 } sf::Mutex
sf::NonCopyable {...} sf::NonCopyable
- myMutexImpl 0x0051b3d8 {myMutex={...} } sf::priv::MutexImpl *
sf::NonCopyable {...} sf::NonCopyable
- myMutex {DebugInfo=0xfeeefeee LockCount=-17891602 RecursionCount=-17891602 ...} _RTL_CRITICAL_SECTION
- DebugInfo 0xfeeefeee {Type=??? CreatorBackTraceIndex=??? CriticalSection=??? ...} _RTL_CRITICAL_SECTION_DEBUG *
Type CXX0030: Error: expression cannot be evaluated
CreatorBackTraceIndex CXX0030: Error: expression cannot be evaluated
CriticalSection CXX0030: Error: expression cannot be evaluated
+ ProcessLocksList {Flink=??? Blink=??? } _LIST_ENTRY
EntryCount CXX0030: Error: expression cannot be evaluated
ContentionCount CXX0030: Error: expression cannot be evaluated
Flags CXX0030: Error: expression cannot be evaluated
CreatorBackTraceIndexHigh CXX0030: Error: expression cannot be evaluated
SpareWORD CXX0030: Error: expression cannot be evaluated
LockCount -17891602 long
RecursionCount -17891602 long
OwningThread 0xfeeefeee void *
LockSemaphore 0xfeeefeee void *
SpinCount 4277075694 unsigned long
Maybe this isn't the same bug........?
-
This one is your fault: you have a global sf::Font instance. This messes up with SFML's globals, and produces the same bug as the default font.
Have you tried to link SFML statically?
There is only one sf::Font, in a class called TextElement, which I create multiple instances of.
Yes SFML is static linked, as this is the only way we can use it without a nightmare of WinSxS issues.
-
There is only one sf::Font, in a class called TextElement, which I create multiple instances of.
...and one of them is destroyed at global exit. Avoiding globals is the best way to avoid problems ;)
-
Hmm.
I might create maybe 8 or 9 of these, and all of them are destroyed at global exit correctly, but the final one causes the crash.
I have modified my SFML library to remove the default font in the constructors, which didn't fix it.
Any other ideas? We might just have to stop using sf::Font altogether if it's causing that much of a problem.
-
Any OpenGL resource that you destroy at global exit will cause problems. Don't do this, really. You should be able to release all your SFML resources before the end of main(), don't you?
-
OK, I am manually destroying these before the program exits, and it has fixed this problem.
Thanks Laurent, I really appreciate your help :)