matlo / GIMX

The GIMX software.
GNU General Public License v3.0
591 stars 105 forks source link

gimx-launcher: An assertion fails when downloading configs #548

Closed amak79 closed 6 years ago

amak79 commented 6 years ago

With commit 63d0b723504ec4ce2710c3e9ab0ec39f6c7914f4, gimx-launcher on Gentoo Linux fails an assertion when downloading configs:

ASSERT INFO:
/var/tmp/portage/x11-libs/wxGTK-3.0.3/work/wxWidgets-3.0.3/src/gtk/gauge.cpp(67): assert "0 <= m_gaugePos && m_gaugePos <= m_rangeMax" failed in DoSetGauge(): invalid gauge position in DoSetGauge()

BACKTRACE:
[1] wxGauge::DoSetGauge()
[2] wxGenericProgressDialog::Update(int, wxString const&, bool*)
[3] curl_multi_perform
[4] curl_easy_perform
[5] configupdater::getconfiglist(char const*)
[6] configupdater::getconfiglist(int (*)(void*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&, unsigned int, unsigned int), void*)
[7] wxEntry(int&, wchar_t**)
[8] __libc_start_main
matlo commented 6 years ago

Thanks for reporting. I think this is due to progressDialog->Update(-1), which is called to check if user has requested cancellation. It seems that the behavior has changed, or that it is dependent on the platform. I can probably provide the last value instead of -1.

amak79 commented 6 years ago

This issue appears to be fixed after the recent commits.