diff options
| author | Albrecht Schlosser <albrechts.fltk@online.de> | 2025-05-30 16:06:56 +0200 |
|---|---|---|
| committer | Albrecht Schlosser <albrechts.fltk@online.de> | 2025-06-05 09:50:07 +0200 |
| commit | 7897bbc04f606417e9d5d91b1cba5aac870e60d9 (patch) | |
| tree | 17c9cac38a6d1b38284e0cd50d7237aadd67d0eb /FL | |
| parent | 2d2f474794f46fae6d12b97661d4bebe5bb21085 (diff) | |
Fix several compiler warnings [-Wextra-semi]
Example:
warning: extra ‘;’ after in-class function definition [-Wextra-semi]
Also: replace 'FL_OVERRIDE' with 'override' in all modified files
Diffstat (limited to 'FL')
| -rw-r--r-- | FL/Fl_PDF_File_Surface.H | 30 | ||||
| -rw-r--r-- | FL/fl_callback_macros.H | 90 |
2 files changed, 60 insertions, 60 deletions
diff --git a/FL/Fl_PDF_File_Surface.H b/FL/Fl_PDF_File_Surface.H index 7fd2ed507..16eabce5c 100644 --- a/FL/Fl_PDF_File_Surface.H +++ b/FL/Fl_PDF_File_Surface.H @@ -1,7 +1,7 @@ // // Declaration of class Fl_PDF_File_Surface for the Fast Light Tool Kit (FLTK). // -// Copyright 2024 by Bill Spitzak and others. +// Copyright 2024-2025 by Bill Spitzak and others. // // This library is free software. Distribution and use rights are outlined in // the file "COPYING" which should have been included with this file. If this @@ -54,7 +54,7 @@ public: */ int begin_job(const char* defaultfilename, char **perr = NULL); /** Don't use for this class */ - int begin_job(int, int *, int *, char **) FL_OVERRIDE {return 1;} + int begin_job(int, int *, int *, char **) override {return 1;} /** Prepare to draw to a PDF document identified by its pathname. \param pathname Path name for the PDF document \param format The paper format for the PDF document @@ -67,23 +67,23 @@ public: enum Fl_Paged_Device::Page_Format format = Fl_Paged_Device::A4, enum Fl_Paged_Device::Page_Layout layout = Fl_Paged_Device::PORTRAIT, char **perr = NULL); - int printable_rect(int *w, int *h) FL_OVERRIDE { return platform_surface_->printable_rect(w,h); } - void margins(int *left, int *top, int *right, int *bottom) FL_OVERRIDE { + int printable_rect(int *w, int *h) override { return platform_surface_->printable_rect(w,h); } + void margins(int *left, int *top, int *right, int *bottom) override { platform_surface_->margins(left,top,right,bottom); } - void origin(int x, int y) FL_OVERRIDE {platform_surface_->origin(x, y);} - void origin(int *x, int *y) FL_OVERRIDE {platform_surface_->origin(x, y);} - void scale(float s_x, float s_y = 0) FL_OVERRIDE {platform_surface_->scale(s_x, s_y);} - void rotate(float angle) FL_OVERRIDE {platform_surface_->rotate(angle);} - void translate(int x, int y) FL_OVERRIDE {platform_surface_->translate(x, y);} - void untranslate() FL_OVERRIDE {platform_surface_->untranslate();}; - int begin_page(void) FL_OVERRIDE {return platform_surface_->begin_page();} - int end_page(void) FL_OVERRIDE {return platform_surface_->end_page();} - void end_job(void) FL_OVERRIDE {return platform_surface_->end_job();} + void origin(int x, int y) override {platform_surface_->origin(x, y);} + void origin(int *x, int *y) override {platform_surface_->origin(x, y);} + void scale(float s_x, float s_y = 0) override {platform_surface_->scale(s_x, s_y);} + void rotate(float angle) override {platform_surface_->rotate(angle);} + void translate(int x, int y) override {platform_surface_->translate(x, y);} + void untranslate() override {platform_surface_->untranslate();} + int begin_page(void) override {return platform_surface_->begin_page();} + int end_page(void) override {return platform_surface_->end_page();} + void end_job(void) override {return platform_surface_->end_job();} /** Returns the name of the PDF document */ inline const char *pdf_filename() { return *out_filename_; } - void set_current() FL_OVERRIDE { if (platform_surface_) platform_surface_->set_current(); } - bool is_current() FL_OVERRIDE { return surface() == platform_surface_; } + void set_current() override { if (platform_surface_) platform_surface_->set_current(); } + bool is_current() override { return surface() == platform_surface_; } }; #endif // PDF_FILE_SURFACE_H diff --git a/FL/fl_callback_macros.H b/FL/fl_callback_macros.H index 398d848b9..48cbdc7b7 100644 --- a/FL/fl_callback_macros.H +++ b/FL/fl_callback_macros.H @@ -1,7 +1,7 @@ /* * Macros for easy callbacks for the Fast Light Tool Kit (FLTK). * - * Copyright 2023 by Bill Spitzak and others. + * Copyright 2023-2025 by Bill Spitzak and others. * * This library is free software. Distribution and use rights are outlined in * the file "COPYING" which should have been included with this file. If this @@ -211,7 +211,7 @@ static void cb(Fl_Widget *w, void *user_data) { Fl_Callback_User_Data_92 *d = (Fl_Callback_User_Data_92*)user_data; hello_2_args_cb(d->p0_, d->p1_); - }; + } Fl_Callback_User_Data_92(const char * p0, int p1) : p0_(p0), p1_(p1) @@ -235,9 +235,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ FUNC(d->p0_, d->p1_, d->p2_, d->p3_, d->p4_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(TYPE0 p0, TYPE1 p1, TYPE2 p2, TYPE3 p3, TYPE4 p4) \ - : p0_(p0), p1_(p1), p2_(p2), p3_(p3), p4_(p4) { }; \ + : p0_(p0), p1_(p1), p2_(p2), p3_(p3), p4_(p4) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(VALUE0, VALUE1, VALUE2, VALUE3, VALUE4), true); \ } while(0) @@ -250,9 +250,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ FUNC(d->p0_, d->p1_, d->p2_, d->p3_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(TYPE0 p0, TYPE1 p1, TYPE2 p2, TYPE3 p3) \ - : p0_(p0), p1_(p1), p2_(p2), p3_(p3) { }; \ + : p0_(p0), p1_(p1), p2_(p2), p3_(p3) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(VALUE0, VALUE1, VALUE2, VALUE3), true); \ } while(0) @@ -265,9 +265,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ FUNC(d->p0_, d->p1_, d->p2_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(TYPE0 p0, TYPE1 p1, TYPE2 p2) \ - : p0_(p0), p1_(p1), p2_(p2) { }; \ + : p0_(p0), p1_(p1), p2_(p2) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(VALUE0, VALUE1, VALUE2), true); \ } while(0) @@ -280,9 +280,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ FUNC(d->p0_, d->p1_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(TYPE0 p0, TYPE1 p1) \ - : p0_(p0), p1_(p1) { }; \ + : p0_(p0), p1_(p1) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(VALUE0, VALUE1), true); \ } while(0) @@ -308,8 +308,8 @@ public: \ static void cb(Fl_Widget *w, void *user_data) { \ FUNC(); \ - }; \ - _FL_CBD_CLASS_NAME() { }; \ + } \ + _FL_CBD_CLASS_NAME() { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(), true); \ } while(0) @@ -341,7 +341,7 @@ static void cb(Fl_Widget *w, void *user_data) { Fl_Callback_User_Data_73 *d = (Fl_Callback_User_Data_73*)user_data; d->self_->resize(d->p0_, d->p1_, d->p2_, d->p3_); - }; + } Fl_Callback_User_Data_73(MyWindow *self, int p0, int p1, int p2, int p3) : self_(self), p0_(p0), p1_(p1), p2_(p2), p3_(p3) { } }; @@ -366,9 +366,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ d->self_->METHOD(d->p0_, d->p1_, d->p2_, d->p3_, d->p4_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(CLASS *self, TYPE0 p0, TYPE1 p1, TYPE2 p2, TYPE3 p3, TYPE4 p4) \ - : self_(self), p0_(p0), p1_(p1), p2_(p2), p3_(p3), p4_(p4) { }; \ + : self_(self), p0_(p0), p1_(p1), p2_(p2), p3_(p3), p4_(p4) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(SELF, VALUE0, VALUE1, VALUE2, VALUE3, VALUE4), true); \ } while(0) @@ -382,9 +382,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ d->self_->METHOD(d->p0_, d->p1_, d->p2_, d->p3_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(CLASS *self, TYPE0 p0, TYPE1 p1, TYPE2 p2, TYPE3 p3) \ - : self_(self), p0_(p0), p1_(p1), p2_(p2), p3_(p3) { }; \ + : self_(self), p0_(p0), p1_(p1), p2_(p2), p3_(p3) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(SELF, VALUE0, VALUE1, VALUE2, VALUE3), true); \ } while(0) @@ -398,9 +398,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ d->self_->METHOD(d->p0_, d->p1_, d->p2_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(CLASS *self, TYPE0 p0, TYPE1 p1, TYPE2 p2) \ - : self_(self), p0_(p0), p1_(p1), p2_(p2) { }; \ + : self_(self), p0_(p0), p1_(p1), p2_(p2) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(SELF, VALUE0, VALUE1, VALUE2), true); \ } while(0) @@ -414,9 +414,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ d->self_->METHOD(d->p0_, d->p1_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(CLASS *self, TYPE0 p0, TYPE1 p1) \ - : self_(self), p0_(p0), p1_(p1) { }; \ + : self_(self), p0_(p0), p1_(p1) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(SELF, VALUE0, VALUE1), true); \ } while(0) @@ -430,9 +430,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ d->self_->METHOD(d->p0_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(CLASS *self, TYPE0 p0) \ - : self_(self), p0_(p0) { }; \ + : self_(self), p0_(p0) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(SELF, VALUE0), true); \ } while(0) @@ -445,9 +445,9 @@ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ d->self_->METHOD(); \ - }; \ + } \ _FL_CBD_CLASS_NAME(CLASS *self) \ - : self_(self) { }; \ + : self_(self) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(SELF), true); \ } while(0) @@ -478,12 +478,12 @@ // lambda style function static void fn(const char * text, int number ) { fl_message("We received the message %s with %d!", text, number); - }; + } // FLTK style callback static void cb(Fl_Widget *w, void *user_data) { Fl_Callback_User_Data_133 *d = (Fl_Callback_User_Data_133*)user_data; fn(d->p0_, d->p1_); - }; + } // class constructor Fl_Callback_User_Data_133(const char * p0, int p1) : p0_(p0), // copy parameter 0 @@ -509,13 +509,13 @@ public: \ TYPE0 p0_; TYPE1 p1_; TYPE2 p2_; TYPE3 p3_; TYPE4 p4_; \ static void fn(TYPE0 NAME0, TYPE1 NAME1, TYPE2 NAME2, TYPE3 NAME3, TYPE4 NAME4) \ - LAMBDA; \ + LAMBDA \ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ _FL_CBD_CLASS_NAME::fn(d->p0_, d->p1_, d->p2_, d->p3_, d->p4_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(TYPE0 p0, TYPE1 p1, TYPE2 p2, TYPE3 p3, TYPE4 p4) \ - : p0_(p0), p1_(p1), p2_(p2), p3_(p3), p4_(p4) { }; \ + : p0_(p0), p1_(p1), p2_(p2), p3_(p3), p4_(p4) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(VALUE0, VALUE1, VALUE2, VALUE3, VALUE4), true); \ } while(0) @@ -526,13 +526,13 @@ public: \ TYPE0 p0_; TYPE1 p1_; TYPE2 p2_; TYPE3 p3_; \ static void fn(TYPE0 NAME0, TYPE1 NAME1, TYPE2 NAME2, TYPE3 NAME3) \ - LAMBDA; \ + LAMBDA \ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ _FL_CBD_CLASS_NAME::fn(d->p0_, d->p1_, d->p2_, d->p3_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(TYPE0 p0, TYPE1 p1, TYPE2 p2, TYPE3 p3) \ - : p0_(p0), p1_(p1), p2_(p2), p3_(p3) { }; \ + : p0_(p0), p1_(p1), p2_(p2), p3_(p3) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(VALUE0, VALUE1, VALUE2, VALUE3), true); \ } while(0) @@ -543,13 +543,13 @@ public: \ TYPE0 p0_; TYPE1 p1_; TYPE2 p2_; \ static void fn(TYPE0 NAME0, TYPE1 NAME1, TYPE2 NAME2) \ - LAMBDA; \ + LAMBDA \ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ _FL_CBD_CLASS_NAME::fn(d->p0_, d->p1_, d->p2_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(TYPE0 p0, TYPE1 p1, TYPE2 p2) \ - : p0_(p0), p1_(p1), p2_(p2) { }; \ + : p0_(p0), p1_(p1), p2_(p2) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(VALUE0, VALUE1, VALUE2), true); \ } while(0) @@ -560,13 +560,13 @@ public: \ TYPE0 p0_; TYPE1 p1_; \ static void fn(TYPE0 NAME0, TYPE1 NAME1) \ - LAMBDA; \ + LAMBDA \ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ _FL_CBD_CLASS_NAME::fn(d->p0_, d->p1_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(TYPE0 p0, TYPE1 p1) \ - : p0_(p0), p1_(p1) { }; \ + : p0_(p0), p1_(p1) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(VALUE0, VALUE1), true); \ } while(0) @@ -577,13 +577,13 @@ public: \ TYPE0 p0_; \ static void fn(TYPE0 NAME0) \ - LAMBDA; \ + LAMBDA \ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME *d = (_FL_CBD_CLASS_NAME*)user_data; \ _FL_CBD_CLASS_NAME::fn(d->p0_); \ - }; \ + } \ _FL_CBD_CLASS_NAME(TYPE0 p0) \ - : p0_(p0) { }; \ + : p0_(p0) { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(VALUE0), true); \ } while(0) @@ -593,11 +593,11 @@ class _FL_CBD_CLASS_NAME : public Fl_Callback_User_Data { \ public: \ static void fn() \ - LAMBDA; \ + LAMBDA \ static void cb(Fl_Widget *w, void *user_data) { \ _FL_CBD_CLASS_NAME::fn(); \ - }; \ - _FL_CBD_CLASS_NAME() { }; \ + } \ + _FL_CBD_CLASS_NAME() { } \ }; \ WIDGET->callback(_FL_CBD_CLASS_NAME::cb, new _FL_CBD_CLASS_NAME(), true); \ } while(0) |
