Closed cschol closed 5 years ago
Thanks, can't test on Linux right now, but I think I found the error (missing NULL declaration to avoid the module browser crash): https://github.com/AScustomWorks/AS/commit/dd1d381866c32cfa33b6af9ddd9d4ff55a61b042
I'll test it. Thanks.
That fixes it. Thanks.
There might be a few couple more BPMcalc: @AScustomWorks
Thread 1 received signal SIGSEGV, Segmentation fault.
0x0000000067eba94e in std::operator<< <char, std::char_traits<char>, std::allocator<char> > (__str=..., __os=...)
at C:/msys64/mingw64/include/c++/8.2.1/bits/basic_string.h:6323
6323 operator<<(basic_ostream<_CharT, _Traits>& __os,
(gdb) bt
#0 0x0000000067eba94e in std::operator<< <char, std::char_traits<char>, std::allocator<char> > (__str=..., __os=...)
at C:/msys64/mingw64/include/c++/8.2.1/bits/basic_string.h:6323
#1 TempodisplayWidget::draw (this=0x2e314cd0, args=...)
at src/BPMCalc.cpp:344
#2 0x00000000004a2388 in rack::widget::Widget::draw (
this=this@entry=0x2e321860, args=...) at src/widget/Widget.cpp:174
#3 0x000000000048e3cb in rack::app::ModuleWidget::draw (this=0x2e321860,
args=...) at src/app/ModuleWidget.cpp:259
#4 0x00000000004a2388 in rack::widget::Widget::draw (this=0x2e3217a0,
args=...) at src/widget/Widget.cpp:174
#5 0x00000000004a2f0f in rack::widget::ZoomWidget::draw (this=0x2e3217a0,
args=...) at src/widget/ZoomWidget.cpp:38
#6 0x00000000004a2388 in rack::widget::Widget::draw (
this=this@entry=0x2e3216a0, args=...) at src/widget/Widget.cpp:174
#7 0x00000000004a3a2b in rack::widget::FramebufferWidget::drawFramebuffer (
this=0x2e3216a0) at src/widget/FramebufferWidget.cpp:187
#8 0x00000000004a32fa in rack::widget::FramebufferWidget::step (
this=0x2e3216a0) at src/widget/FramebufferWidget.cpp:79
#9 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d461c0)
at src/widget/Widget.cpp:151
#10 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d46120)
at src/widget/Widget.cpp:151
#11 0x00000000004a2bd7 in rack::widget::Widget::step (
this=this@entry=0x6d3cb60) at src/widget/Widget.cpp:151
#12 0x00000000004a11fb in rack::ui::SequentialLayout::step (this=0x6d3cb60)
at src/ui/SequentialLayout.cpp:14
#13 0x00000000004a2bd7 in rack::widget::Widget::step (
this=this@entry=0x6d3ca90) at src/widget/Widget.cpp:151
#14 0x000000000049f0dd in rack::ui::MarginLayout::step (this=0x6d3ca90)
at src/ui/MarginLayout.cpp:10
#15 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d3c7e0)
at src/widget/Widget.cpp:151
#16 0x00000000004a2bd7 in rack::widget::Widget::step (
this=this@entry=0x6d3c750) at src/widget/Widget.cpp:151
#17 0x000000000049ead3 in rack::ui::ScrollWidget::step (this=0x6d3c750)
at src/ui/ScrollWidget.cpp:36
#18 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d3bd50)
at src/widget/Widget.cpp:151
#19 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6d3bce0)
at src/widget/Widget.cpp:151
#20 0x00000000004a2bd7 in rack::widget::Widget::step (this=0x6cc70d0)
at src/widget/Widget.cpp:151
#21 0x000000000048e0c4 in rack::app::Scene::step (this=0x6cc70d0)
at src/app/Scene.cpp:51
#22 0x0000000000454bf0 in rack::Window::run (this=0x6cd3450)
at src/window.cpp:363
#23 0x000000000079c7dd in main (argc=<optimized out>, argv=<optimized out>)
at src/main.cpp:175
(gdb) make: *** [Makefile:64: debug] Interrupt
@Coirt Is this in the module browser or when the module is running?
Browser
Confirmed. It is seemingly timing related whether the crash occurs. A few times I got it to work...but it is definitely an issue. Thanks for reporting.
This should do it:
diff --git a/src/BPMCalc.cpp b/src/BPMCalc.cpp
index f8a353a..30d15f4 100755
--- a/src/BPMCalc.cpp
+++ b/src/BPMCalc.cpp
@@ -312,7 +312,7 @@ struct BPMCalc : Module {
////////////////////////////////////
struct TempodisplayWidget : TransparentWidget {
- std::string *value;
+ std::string *value = NULL;
std::shared_ptr<Font> font;
TempodisplayWidget() {
@@ -634,4 +634,4 @@ struct BPMCalcWidget : ModuleWidget {
};
-Model *modelBPMCalc = createModel<BPMCalc, BPMCalcWidget>("BPMCalc");
\ No newline at end of file
+Model *modelBPMCalc = createModel<BPMCalc, BPMCalcWidget>("BPMCalc");
diff --git a/src/BPMCalc2.cpp b/src/BPMCalc2.cpp
index 62d9d2d..08995a0 100755
--- a/src/BPMCalc2.cpp
+++ b/src/BPMCalc2.cpp
@@ -312,7 +312,7 @@ struct BPMCalc2 : Module {
////////////////////////////////////
struct TempodisplayWidget : TransparentWidget {
- std::string *value;
+ std::string *value = NULL;
std::shared_ptr<Font> font;
TempodisplayWidget() {
@@ -425,4 +425,4 @@ struct BPMCalc2Widget : ModuleWidget {
};
-Model *modelBPMCalc2 = createModel<BPMCalc2, BPMCalc2Widget>("BPMCalc2");
\ No newline at end of file
+Model *modelBPMCalc2 = createModel<BPMCalc2, BPMCalc2Widget>("BPMCalc2");
Interesting, yeah confirmed now loading! :sake:
Oh, missed those two. fixed now: https://github.com/AScustomWorks/AS/commit/15e89652b788e0c4c17b1a74377628a98424ea5e And added "Distortion" tag to the Overdrive FX module in plugin.json https://github.com/AScustomWorks/AS/commit/730c76405a5f21c817df62c90537484cfa83b8a7
Thanks!
Seems fixed :+1:
Backtrace on Ubuntu 18.04: