cppcheck fixes discovered by Andrew
authorGood Guy <[email protected]>
Sat, 4 Jun 2022 01:04:53 +0000 (19:04 -0600)
committerGood Guy <[email protected]>
Sat, 4 Jun 2022 01:04:53 +0000 (19:04 -0600)
cinelerra-5.1/cinelerra/cropvideo.C
cinelerra-5.1/cinelerra/dbwindow.C
cinelerra-5.1/cinelerra/formatwindow.C
cinelerra-5.1/cinelerra/pluginmessages.h
cinelerra-5.1/cinelerra/pluginprefs.C
cinelerra-5.1/cinelerra/recordengine.C
cinelerra-5.1/cinelerra/scale.C

index 72aa0b9d96f0f390828c77cf07f89bbd80bc8080..08adc4851c6429133dd43f3091d75a36efa0e54b 100644 (file)
@@ -87,10 +87,12 @@ void CropVideo::run()
 
 int CropVideo::load_defaults()
 {
+return 0;
 }
 
 int CropVideo::save_defaults()
 {
+return 0;
 }
 
 CropVideoWindow::CropVideoWindow(MWindow *mwindow, CropVideo *thread)
index 980702345f2d48af4ec2a79f3b4238c9bf6fdac1..4bf9d0ca2127aedcc48c93cef16237063ebc6e00 100644 (file)
@@ -840,8 +840,8 @@ DbWindowItem(int id, const char *source, const char *title,
 DbWindowItem::
 ~DbWindowItem()
 {
-       delete source;
-       delete title;
+       delete [] source;
+       delete [] title;
 }
 
 #define CmprFn(nm,key) int DbWindowGUI:: \
index 67f7416df1d29c6308e7bc0a3f0795cfb49058bb..c3726c3c8edb4a448d7afe417dc0f7a1c3623b08 100644 (file)
@@ -142,6 +142,7 @@ FormatQuality::~FormatQuality()
 int FormatQuality::handle_event()
 {
        asset->quality = get_value();
+return 0;
 }
 
 
@@ -152,6 +153,7 @@ FormatBits::~FormatBits() {}
 int FormatBits::handle_event()
 {
        asset->bits = get_bits();
+return 0;
 }
 
 
@@ -162,6 +164,7 @@ FormatDither::~FormatDither() {}
 int FormatDither::handle_event()
 {
        *dither = get_value();
+return 0;
 }
 
 
@@ -174,6 +177,7 @@ FormatSigned::~FormatSigned() {}
 int FormatSigned::handle_event()
 {
        asset->signed_ = get_value();
+return 0;
 }
 
 
@@ -188,6 +192,7 @@ int FormatHILO::handle_event()
 {
        asset->byte_order = get_value() ^ 1;
        lohi->update(get_value() ^ 1);
+return 0;
 }
 
 FormatLOHI::FormatLOHI(int x, int y, FormatHILO *hilo, Asset *asset)
@@ -202,5 +207,6 @@ int FormatLOHI::handle_event()
 {
        asset->byte_order = get_value();
        hilo->update(get_value() ^ 1);
+return 0;
 }
 
index f45c9bd1c6489fa76ed4cf3f4b7aa037730123c9..0a63dd907b12612da54ec451c7d025a68df9dc1e 100644 (file)
@@ -32,19 +32,19 @@ public:
        PluginMessages(int input_flag, int output_flag, int message_id = -1);
        ~PluginMessages();
 
-       send_message(char *text);
-       receive_message(char *text);
-
-       send_message(int command, char *text);
-       send_message(long command, long value);
-       send_message(long command, long value1, long value2);
-       send_message(int command);
-
-       receive_message();     // returns the command
-       receive_message(int *command, char *text);
-       receive_message(int *command, long *value);
-       receive_message(long *value1, long *value2);
-       receive_message(int *command, long *value1, long *value2);
+       void send_message(char *text);
+       void receive_message(char *text);
+
+       void send_message(int command, char *text);
+       void send_message(long command, long value);
+       void send_message(long command, long value1, long value2);
+       void send_message(int command);
+
+       void receive_message();     // returns the command
+       void receive_message(int *command, char *text);
+       void receive_message(int *command, long *value);
+       void receive_message(long *value1, long *value2);
+       void receive_message(int *command, long *value1, long *value2);
 
        Messages *messages;
        int input_flag, output_flag;
index 2a3f7878cc27760fa43232d5f447084335062e4b..b8167bb2378628a6e28f70e4dd2a520e54c86303 100644 (file)
@@ -93,6 +93,7 @@ PluginGlobalPathText::~PluginGlobalPathText() {}
 int PluginGlobalPathText::handle_event()
 {
        strcpy(pwindow->thread->preferences->global_plugin_dir, get_text());
+return 0;
 }
 
 
@@ -110,4 +111,5 @@ PluginLocalPathText::~PluginLocalPathText() {}
 int PluginLocalPathText::handle_event()
 {
        strcpy(pwindow->thread->preferences->local_plugin_dir, get_text());
+return 0;
 }
index 2fb1489115e832845f8454f916879c04f07b5156..9e604596a3bc60675d4450607843c173ca2ece60 100644 (file)
@@ -127,6 +127,7 @@ int RecordEngine::initialize()
        current_jump_jumps[2] = 60;
        current_jump_jumps[3] = 80;
        current_jump_jumps[4] = 100;
+return 0;
 }
 
 int RecordEngine::run_script(FileXML *script)
@@ -199,6 +200,7 @@ long RecordEngine::get_dc_offset(int offset)
 int RecordEngine::set_dc_offset(long new_offset, int number)
 {
        adevice->set_dc_offset(new_offset, number);
+return 0;
 }
 
 long int RecordEngine::get_dc_offset(long *dc_offset, RecordGUIDCOffsetText **dc_offset_text)
@@ -210,6 +212,7 @@ int RecordEngine::set_gui(RecordGUI *gui)
 {
        this->gui = gui;
        update_position(current_position);
+return 0;
 }
 
 int RecordEngine::get_duplex_enable()
@@ -491,11 +494,13 @@ int RecordEngine::close_output_devices()
 int RecordEngine::lock_window()
 {
        gui->lock_window();
+return 0;
 }
 
 int RecordEngine::unlock_window()
 {
        gui->unlock_window();
+return 0;
 }
 
 int RecordEngine::update_position(long new_position)
@@ -523,6 +528,7 @@ int RecordEngine::update_position(long new_position)
 
                gui->update_next_label(next_label);
        }
+return 0;
 }
 
 int RecordEngine::goto_prev_label()
@@ -540,6 +546,7 @@ int RecordEngine::goto_prev_label()
                        update_position(new_position);
                }
        }
+return 0;
 }
 
 int RecordEngine::goto_next_label()
@@ -588,6 +595,7 @@ int RecordEngine::calibrate_dc_offset(long new_value, int channel)
 
 int RecordEngine::reset_over()
 {
+return 0;
 }
 
 int RecordEngine::set_done(int value)
@@ -595,6 +603,7 @@ int RecordEngine::set_done(int value)
        stop_operation(1);
        stop_monitor();
        gui->set_done(value);
+return 0;
 }
 
 int RecordEngine::start_over()
@@ -623,6 +632,7 @@ int RecordEngine::start_over()
                        record->startsource_frame = 0;
                }
        }
+return 0;
 }
 
 int RecordEngine::change_channel(Channel *channel)
@@ -648,6 +658,7 @@ int RecordEngine::get_format(char *string)
 {
        File file;
        strcpy(string, file.formattostr(mwindow->plugindb, asset->format));
+       return 0;
 }
 int RecordEngine::get_samplerate() { return asset->rate; }
 int RecordEngine::get_bits() { return asset->bits; }
@@ -665,25 +676,30 @@ float RecordEngine::get_frames_per_foot() { /* return mwindow->preferences->fram
 
 int RecordEngine::set_monitor_video(int value)
 {
+return 0;
 }
 
 int RecordEngine::set_monitor_audio(int value)
 {
+return 0;
 }
 
 int RecordEngine::set_record_mode(char *text)
 {
        record->record_mode = text_to_mode(text);
+return 0;
 }
 
 int RecordEngine::get_record_mode(char *text)
 {
        mode_to_text(text, record->record_mode);
+return 0;
 }
 
 int RecordEngine::get_record_mode()
 {
        return record->record_mode;
+return 0;
 }
 
 int RecordEngine::mode_to_text(char *string, int mode)
@@ -694,6 +710,7 @@ int RecordEngine::mode_to_text(char *string, int mode)
                case 1:        sprintf(string, _("Timed"));         break;
                case 2:        sprintf(string, _("Loop"));          break;
        }
+return 0;
 }
 
 int RecordEngine::text_to_mode(char *string)
@@ -701,6 +718,7 @@ int RecordEngine::text_to_mode(char *string)
        if(!strcasecmp(string, _("Untimed"))) return 0;
        if(!strcasecmp(string, _("Timed")))   return 1;
        if(!strcasecmp(string, _("Loop")))    return 2;
+return 0;
 }
 
 long RecordEngine::get_current_delay()
@@ -718,11 +736,13 @@ int RecordEngine::reset_current_delay()
 {
        current_jump_delay = 0;
        current_jump_jump = current_jump_jumps[current_jump_delay];
+return 0;
 }
 
 int RecordEngine::set_loop_duration()
 {
        record->set_loop_duration((long)record->get_samplerate() * (atol(gui->loop_sec->get_text()) + atol(gui->loop_min->get_text()) * 60 + atol(gui->loop_hr->get_text()) * 3600));
+return 0;
 }
 
 
index 860c46fb0e010014805e41ed5bccb86eb2c9b3fe..7d682bcb1706fd20dc15d9cf26d9c660fc4adb3b 100644 (file)
@@ -47,6 +47,7 @@ Scale::~Scale()
 int Scale::handle_event()
 {
        thread->start();
+return 0;
 }
 
 ScaleThread::ScaleThread(MWindow *mwindow)
@@ -189,6 +190,7 @@ int ScaleThread::update_aspect(ScaleWindow *window)
                sprintf(string, "%.0f", aspect_h);
                window->aspect_h->update(string);
        }
+return 0;
 }
 
 
@@ -307,6 +309,7 @@ int ScaleSizeText::handle_event()
        if(*output > 10000) *output = 10000;
        *output *= -1;
        thread->update_window();
+return 0;
 }
 
 ScaleOffsetText::ScaleOffsetText(int x, int y, ScaleThread *thread, int *output)
@@ -320,6 +323,7 @@ int ScaleOffsetText::handle_event()
        if(*output > 10000) *output = 10000;
        if(*output < -10000) *output = -10000;
        thread->update_window(1);
+return 0;
 }
 
 ScaleRatioText::ScaleRatioText(int x, int y, ScaleThread *thread, float *output)
@@ -334,6 +338,7 @@ int ScaleRatioText::handle_event()
        if(*output < -10000) *output = -10000;
        *output *= -1;
        thread->update_window();
+return 0;
 }
 
 
@@ -346,6 +351,7 @@ ScaleConstrain::~ScaleConstrain() {}
 int ScaleConstrain::handle_event()
 {
        thread->constrain_ratio = get_value();
+return 0;
 }
 
 ScaleData::ScaleData(int x, int y, ScaleThread *thread)
@@ -356,6 +362,7 @@ int ScaleData::handle_event()
 {
        thread->scale_data = get_value();
        thread->update_window();
+return 0;
 }
 
 
@@ -371,6 +378,7 @@ int ScaleAspectAuto::handle_event()
 {
        thread->auto_aspect = get_value();
        thread->update_aspect(thread->window);
+return 0;
 }
 
 
@@ -389,6 +397,7 @@ ScaleAspectW::~ScaleAspectW()
 int ScaleAspectW::handle_event()
 {
        *output = atof(get_text());
+return 0;
 }
 
 
@@ -405,6 +414,7 @@ ScaleAspectH::~ScaleAspectH()
 int ScaleAspectH::handle_event()
 {
        *output = atof(get_text());
+return 0;
 }