Warnings: Difference between revisions

From vice-emu
Jump to navigation Jump to search
VisualWikitext
Line 4: Line 4:




* gpz (x86, Linux/Gentoo, 64bit, GCC 7.3.0, GTK3) (r35613)
* gpz (x86, Linux/Gentoo, 64bit, GCC 7.3.0, GTK3) (r35655)


<pre>
<pre>
Line 15: Line 15:
             case P_READY:
             case P_READY:
             ^~~~
             ^~~~
Making all in parallel
Making all in tape
Making all in imagecontents
Making all in fileio
Making all in video
Making all in raster
Making all in vicii
Making all in viciisc
Making all in viciivsid
Making all in vdc
Making all in crtc
Making all in gfxoutputdrv
Making all in printerdrv
Making all in rs232drv
Making all in sid
Making all in monitor
Making all in core
Making all in core
viacore.c: In function 'viacore_read':
viacore.c: In function 'viacore_read':
Line 47: Line 31:
         case DS12C887_REG_CTRL_A:
         case DS12C887_REG_CTRL_A:
         ^~~~
         ^~~~
Making all in userport
Making all in tapeport
Making all in diag
Making all in c64
Making all in cart
Making all in scpu64
Making all in c64dtv
c64dtvcpu.c:537:2: warning: #warning "CPUMEMHISTORY implementation for x64dtv is incomplete" [-Wcpp]
#warning "CPUMEMHISTORY implementation for x64dtv is incomplete"
  ^~~~~~~
Making all in c128
Making all in c128
c128cpu.c:123:2: warning: #warning "CPUMEMHISTORY implementation for x128 is incomplete" [-Wcpp]
#warning "CPUMEMHISTORY implementation for x128 is incomplete"
  ^~~~~~~
functionrom.c: In function 'set_external_function_rom_enabled':
functionrom.c: In function 'set_external_function_rom_enabled':
functionrom.c:154:26: warning: this statement may fall through [-Wimplicit-fallthrough=]
functionrom.c:154:26: warning: this statement may fall through [-Wimplicit-fallthrough=]
Line 75: Line 46:
         case INT_FUNCTION_RAM:
         case INT_FUNCTION_RAM:
         ^~~~
         ^~~~
Making all in vic20
Making all in cart
Making all in pet
Making all in pet
petcpu.c:69:2: warning: #warning "CPUMEMHISTORY implementation for xpet is incomplete" [-Wcpp]
#warning "CPUMEMHISTORY implementation for xpet is incomplete"
  ^~~~~~~
petmem.c: In function 'read_io_e8':
petmem.c: In function 'read_io_e8':
petmem.c:770:16: warning: this statement may fall through [-Wimplicit-fallthrough=]
petmem.c:770:16: warning: this statement may fall through [-Wimplicit-fallthrough=]
Line 88: Line 54:
         case 0x00:
         case 0x00:
         ^~~~
         ^~~~
petmemsnapshot.c: In function 'mem_read_ram_snapshot_module':
petmemsnapshot.c:211:9: warning: declaration of 'spet_bank' shadows a global declaration [-Wshadow]
    int spet_bank = 0;
        ^~~~~~~~~
In file included from petmemsnapshot.c:43:0:
petmem.h:79:12: note: shadowed declaration is here
extern int spet_bank;
            ^~~~~~~~~
petprinter.c: In function 'machine_printer_setup_context':
petprinter.c:35:62: warning: declaration of 'machine_context' shadows a global declaration [-Wshadow]
void machine_printer_setup_context(struct machine_context_s *machine_context)
                                                              ^~~~~~~~~~~~~~~
In file included from petprinter.c:30:0:
pet.h:92:26: note: shadowed declaration is here
extern machine_context_t machine_context;
                          ^~~~~~~~~~~~~~~
petrom.c: In function 'mem_load':
petrom.c:850:13: warning: declaration of 'i' shadows a previous local [-Wshadow]
        int i;
            ^
petrom.c:797:9: note: shadowed declaration is here
    int i;
        ^
petvia.c: In function 'petvia_setup_context':
petvia.c:238:46: warning: declaration of 'machine_context' shadows a global declaration [-Wshadow]
void petvia_setup_context(machine_context_t *machine_context)
                                              ^~~~~~~~~~~~~~~
In file included from petvia.c:43:0:
pet.h:92:26: note: shadowed declaration is here
extern machine_context_t machine_context;
                          ^~~~~~~~~~~~~~~
Making all in plus4
plus4cpu.c:40:2: warning: #warning "CPUMEMHISTORY implementation for xplus4 is incomplete" [-Wcpp]
#warning "CPUMEMHISTORY implementation for xplus4 is incomplete"
  ^~~~~~~
Making all in cbm2
cbm2cpu.c:68:2: warning: #warning "CPUMEMHISTORY implementation for xcbm2 is incomplete" [-Wcpp]
#warning "CPUMEMHISTORY implementation for xcbm2 is incomplete"
  ^~~~~~~
cbm5x0cia1.c: In function 'cia1_setup_context':
cbm5x0cia1.c:220:44: warning: declaration of 'machine_context' shadows a global declaration [-Wshadow]
void cia1_setup_context(machine_context_t *machine_context)
                                            ^~~~~~~~~~~~~~~
In file included from cbm5x0cia1.c:39:0:
cbm2.h:98:26: note: shadowed declaration is here
extern machine_context_t machine_context;
                          ^~~~~~~~~~~~~~~
Making all in arch/shared
Making all in arch
Making all in gtk3
Making all in data
Making all in novte
Making all in widgets
Making all in base
Making all in shared
Making all in lib
Making all in p64
Making all in hvsc
Making all in hvsc
datasette.c: In function 'datasette_control_internal':
datasette.c: In function 'datasette_control_internal':
Line 159: Line 68:
             case DATASETTE_CONTROL_STOP:    /* fall through */
             case DATASETTE_CONTROL_STOP:    /* fall through */
             ^~~~
             ^~~~
</pre>
</pre>


Revision as of 00:23, 18 October 2018

GCC

  • gpz (x86, Linux/Gentoo, 64bit, GCC 7.3.0, GTK3) (r35655)
Making all in serial
serial-iec-device.c: In function 'serial_iec_device_exec_main':
serial-iec-device.c:641:20: warning: this statement may fall through [-Wimplicit-fallthrough=]
                 if (clk_value >= iec->timeout) {
                    ^
serial-iec-device.c:649:13: note: here
             case P_READY:
             ^~~~
Making all in core
viacore.c: In function 'viacore_read':
viacore.c:721:16: warning: this statement may fall through [-Wimplicit-fallthrough=]
             if (via_context->ier & (VIA_IM_CA1 | VIA_IM_CA2)) {
                ^
viacore.c:725:9: note: here
         case VIA_PRA_NHS: /* port A, no handshake */
         ^~~~
Making all in rtc
ds12c887.c: In function 'ds12c887_store_data':
ds12c887.c:658:16: warning: this statement may fall through [-Wimplicit-fallthrough=]
             if (context->set) {
                ^
ds12c887.c:664:9: note: here
         case DS12C887_REG_CTRL_A:
         ^~~~
Making all in c128
functionrom.c: In function 'set_external_function_rom_enabled':
functionrom.c:154:26: warning: this statement may fall through [-Wimplicit-fallthrough=]
             rtc2_context = bq4830y_init("EFR");
             ~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
functionrom.c:155:9: note: here
         case EXT_FUNCTION_RAM:
         ^~~~
functionrom.c: In function 'set_internal_function_rom_enabled':
functionrom.c:103:26: warning: this statement may fall through [-Wimplicit-fallthrough=]
             rtc1_context = bq4830y_init("IFR");
             ~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
functionrom.c:104:9: note: here
         case INT_FUNCTION_RAM:
         ^~~~
Making all in pet
petmem.c: In function 'read_io_e8':
petmem.c:770:16: warning: this statement may fall through [-Wimplicit-fallthrough=]
             if (petres.crtc) {
                ^
petmem.c:773:9: note: here
         case 0x00:
         ^~~~
Making all in hvsc
datasette.c: In function 'datasette_control_internal':
datasette.c:859:17: warning: this statement may fall through [-Wimplicit-fallthrough=]
                 datasette_internal_reset();
                 ^~~~~~~~~~~~~~~~~~~~~~~~~~
datasette.c:860:13: note: here
             case DATASETTE_CONTROL_STOP:    /* fall through */
             ^~~~
datasette.c:916:17: warning: this statement may fall through [-Wimplicit-fallthrough=]
                 datasette_internal_reset();
                 ^~~~~~~~~~~~~~~~~~~~~~~~~~
datasette.c:917:13: note: here
             case DATASETTE_CONTROL_STOP:    /* fall through */
             ^~~~


  • gpz (x86, Linux/Gentoo, 64bit, GCC 6.4.0, GTK3) (3.2 tarball)
./configure --program-prefix=gtk3- --enable-debug --enable-native-gtk3ui --enable-debug-gtk3ui --enable-native-tools --enable-arch=native --prefix=/usr/games/ --enable-parsid --with-midas --with-resid --with-xaw3d --with-arts --with-x --enable-ethernet --with-pulse --with-sdlsound --enable-fullscreen --enable-cpuhistory --enable-static-ffmpeg --enable-static-lame

no warnings besides a bunch of ffmpeg related things that are not listed here (we dont fix them)


  • compyx (x64, Windows 10/msys2, GCC 7.3.0, SDL2) (r35061)
Quite a few warnings after updating msys64, resulting in upgrading GCC from 7.3.0 to 8.2.0, though we can probably ignore the warnings about VTE.
In file included from ../../../../vice/src/arch/sdl/archdep.c:54:0:
../../../../vice/src/arch/sdl/archdep_win32.c: In function 'kbd_arch_get_host_mapping':
../../../../vice/src/arch/sdl/archdep_win32.c:781:16: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
     int lang = (int)GetKeyboardLayout(0);
                ^
At top level:
../../../../vice/src/arch/sdl/archdep_win32.c:224:13: warning: 'system_wcstombs_free' defined but not used [-Wunused-function]
 static void system_wcstombs_free(char *mbs)
             ^22:48, 18 June 2018 (CEST)22:48, 18 June 2018 (CEST)22:48, 18 June 2018 (CEST)[[User:Compyx|Compyx]] ([[User talk:Compyx|talk]]) 22:48, 18 June 2018 (CEST)
../../../../vice/src/arch/sdl/archdep_win32.c:210:14: warning: 'system_wcstombs_alloc' defined but not used [-Wunused-function]
 static char *system_wcstombs_alloc(const char *wcs)
              ^22:48, 18 June 2018 (CEST)22:48, 18 June 2018 (CEST)22:48, 18 June 2018 (CEST)22:48, 18 June 2018 (CEST)
../../../../vice/src/arch/sdl/menu_c64cart.c: In function 'makegroup':
../../../../vice/src/arch/sdl/menu_c64cart.c:87:37: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
             ui_c64cart_entry.data = (ui_callback_data_t)(unsigned long)cartlist->crtid;
                                     ^
In file included from ../../../../vice/src/arch/sdl/menu_midi.c:35:0:
../../../../vice/src/arch/sdl/menu_midi.c:161:18: warning: no previous prototype for 'MIDIInDev_dynmenu_callback' [-Wmissing-prototypes]
 UI_MENU_CALLBACK(MIDIInDev_dynmenu_callback)
                  ^
../../../../vice/src/arch/sdl/menu_common.h:62:17: note: in definition of macro 'UI_MENU_CALLBACK'
     const char *name(int activated, ui_callback_data_t param)
                 ^[[User:Compyx|Compyx]] ([[User talk:Compyx|talk]])
../../../../vice/src/arch/sdl/menu_midi.c: In function 'MIDIInDev_dynmenu_callback':
../../../../vice/src/arch/sdl/menu_midi.c:189:44: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
                 midi_in_dyn_menu[i].data = (ui_callback_data_t)j;
                                            ^
In file included from ../../../../vice/src/arch/sdl/menu_midi.c:35:0:
../../../../vice/src/arch/sdl/menu_midi.c: At top level:
../../../../vice/src/arch/sdl/menu_midi.c:203:18: warning: no previous prototype for 'MIDIOutDev_dynmenu_callback' [-Wmissing-prototypes]
 UI_MENU_CALLBACK(MIDIOutDev_dynmenu_callback)
                  ^
../../../../vice/src/arch/sdl/menu_common.h:62:17: note: in definition of macro 'UI_MENU_CALLBACK'
     const char *name(int activated, ui_callback_data_t param)
                 ^[[User:Compyx|Compyx]] ([[User talk:Compyx|talk]])
../../../../vice/src/arch/sdl/menu_midi.c: In function 'MIDIOutDev_dynmenu_callback':
../../../../vice/src/arch/sdl/menu_midi.c:231:45: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
                 midi_out_dyn_menu[i].data = (ui_callback_data_t)j;
                                             ^
../../../../vice/src/arch/sdl/menu_settings.c: In function 'uikeyboard_menu_create':
../../../../vice/src/arch/sdl/menu_settings.c:179:42: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
         ui_keyboard_mapping_entry.data = (ui_callback_data_t)(unsigned long)kbdlist->mapping;
                                         ^


  • compyx (x64, Windows 10/msys2, GCC 8.2.0, Gtk3) (r35433)
Making all in hwsiddrv
../../../vice/src/hwsiddrv/ps-win32-dll.c: In function 'parsid_GetAddressLptPortInTheRegistry':
../../../vice/src/hwsiddrv/ps-win32-dll.c:243:26: warning: '%d' directive writing 1 byte into a region of size between 0 and 254 [-Wformat-overflow=]
     sprintf(myData, "%s\\%d", myKey, myPort - 1);
                          ^~
../../../vice/src/hwsiddrv/ps-win32-dll.c:243:21: note: directive argument in the range [0, 2]
     sprintf(myData, "%s\\%d", myKey, myPort - 1);
                     ^~~~~~~~
../../../vice/src/hwsiddrv/ps-win32-dll.c:243:5: note: 'sprintf' output between 3 and 257 bytes into a destination of size 255
     sprintf(myData, "%s\\%d", myKey, myPort - 1);
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../../vice/src/hwsiddrv/ps-win32-io.c: In function 'parsid_GetAddressLptPortInTheRegistry':
../../../vice/src/hwsiddrv/ps-win32-io.c:207:26: warning: '%d' directive writing 1 byte into a region of size between 0 and 254 [-Wformat-overflow=]
     sprintf(myData, "%s\\%d", myKey, myPort - 1);
                          ^~
../../../vice/src/hwsiddrv/ps-win32-io.c:207:21: note: directive argument in the range [0, 2]
     sprintf(myData, "%s\\%d", myKey, myPort - 1);
                     ^~~~~~~~
../../../vice/src/hwsiddrv/ps-win32-io.c:207:5: note: 'sprintf' output between 3 and 257 bytes into a destination of size 255
     sprintf(myData, "%s\\%d", myKey, myPort - 1);
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~



Making all in cart
../../../../vice/src/c64/cart/crt.c: In function 'crt_open':
../../../../vice/src/c64/cart/crt.c:160:9: warning: 'strncpy' output may be truncated copying 32 bytes from a string of length 63 [-Wstringop-truncation]
         strncpy(header->name, (char*)&crt_header[0x20], sizeof(header->name) - 1);
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../../../vice/src/c64/cart/crt.c: In function 'crt_create':
../../../../vice/src/c64/cart/crt.c:289:5: warning: 'strncpy' specified bound 32 equals destination size [-Wstringop-truncation]
     strncpy((char*)&crt_header[0x20], name, sizeof(crt_header) - 0x20);
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Making all in scpu64
In file included from ../../../vice/src/scpu64/../main65816cpu.c:341,
                 from ../../../vice/src/scpu64/scpu64cpu.c:431:
../../../vice/src/scpu64/../main65816cpu.c: In function 'maincpu_mainloop':
../../../vice/src/scpu64/../65816core.c:3648:39: warning: 'p3' may be used uninitialized in this function [-Wmaybe-uninitialized]
             uint8_t bk = (uint8_t)(p3 >> 16);
                                   ~~~~^~~~~~
../../../vice/src/scpu64/../65816core.c:2554:34: note: 'p3' was declared here
         unsigned int p0, p1, p2, p3;
                                  ^~
../../../vice/src/scpu64/../65816core.c:3647:39: warning: 'p2' may be used uninitialized in this function [-Wmaybe-uninitialized]
             uint8_t hi = (uint8_t)(p2 >> 8);
                                   ~~~~^~~~~
../../../vice/src/scpu64/../65816core.c:2554:30: note: 'p2' was declared here
         unsigned int p0, p1, p2, p3;
                              ^~
../../../vice/src/scpu64/../65816core.c:3650:13: warning: 'p1' may be used uninitialized in this function [-Wmaybe-uninitialized]
             debug_main65816cpu((uint32_t)(debug_pc), debug_clk,
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                           mon_disassemble_to_string(e_comp_space,
                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                                     debug_pc, op,
                                                     ~~~~~~~~~~~~~
                                                     lo, hi, bk, 1, "65816"),
                                                     ~~~~~~~~~~~~~~~~~~~~~~~~
                           debug_c, debug_x, debug_y, debug_sp, debug_pbr);
                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../../vice/src/scpu64/../65816core.c:2554:26: note: 'p1' was declared here
         unsigned int p0, p1, p2, p3;
                          ^~



Making all in novte
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:100,
                 from ../../../../../vice/src/arch/gtk3/novte/keymap.cc:26:
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:66:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:67:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:68:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:69:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:271,
                 from ../../../../../vice/src/arch/gtk3/novte/keymap.cc:26:
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:74:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:75:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:76:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:77:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:100,
                 from ../../../../../vice/src/arch/gtk3/novte/vteterminal.h:25,
                 from ../../../../../vice/src/arch/gtk3/novte/novte.h:28,
                 from ../../../../../vice/src/arch/gtk3/novte/ring.h:27,
                 from ../../../../../vice/src/arch/gtk3/novte/ring.cc:24:
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:66:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:67:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:68:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:69:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:271,
                 from ../../../../../vice/src/arch/gtk3/novte/vteterminal.h:25,
                 from ../../../../../vice/src/arch/gtk3/novte/novte.h:28,
                 from ../../../../../vice/src/arch/gtk3/novte/ring.h:27,
                 from ../../../../../vice/src/arch/gtk3/novte/ring.cc:24:
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:74:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:75:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:76:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:77:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:100,
                 from ../../../../../vice/src/arch/gtk3/novte/vteterminal.h:25,
                 from ../../../../../vice/src/arch/gtk3/novte/novte.h:28,
                 from ../../../../../vice/src/arch/gtk3/novte/vte.cc:35:
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:66:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:67:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:68:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:69:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:271,
                 from ../../../../../vice/src/arch/gtk3/novte/vteterminal.h:25,
                 from ../../../../../vice/src/arch/gtk3/novte/novte.h:28,
                 from ../../../../../vice/src/arch/gtk3/novte/vte.cc:35:
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:74:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:75:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:76:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:77:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:100,
                 from ../../../../../vice/src/arch/gtk3/novte/vteaccess.cc:32:
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:66:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:67:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:68:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:69:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:271,
                 from ../../../../../vice/src/arch/gtk3/novte/vteaccess.cc:32:
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:74:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:75:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:76:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:77:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:100,
                 from ../../../../../vice/src/arch/gtk3/novte/vtedraw.cc:33:
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:66:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:67:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:68:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:69:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:271,
                 from ../../../../../vice/src/arch/gtk3/novte/vtedraw.cc:33:
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:74:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:75:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:76:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:77:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:100,
                 from ../../../../../vice/src/arch/gtk3/novte/vtegtk.cc:44:
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:66:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:67:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:68:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:69:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:271,
                 from ../../../../../vice/src/arch/gtk3/novte/vtegtk.cc:44:
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:74:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:75:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:76:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:77:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:100,
                 from ../../../../../vice/src/arch/gtk3/novte/vteterminal.h:25,
                 from ../../../../../vice/src/arch/gtk3/novte/novte.h:28,
                 from ../../../../../vice/src/arch/gtk3/novte/vteseq.cc:33:
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:66:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:67:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:68:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/gtkfilechooserbutton.h:69:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^
In file included from C:/msys64/mingw64/include/gtk-3.0/gtk/gtk.h:271,
                 from ../../../../../vice/src/arch/gtk3/novte/vteterminal.h:25,
                 from ../../../../../vice/src/arch/gtk3/novte/novte.h:28,
                 from ../../../../../vice/src/arch/gtk3/novte/vteseq.cc:33:
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:74:8: warning: unnecessary parentheses in declaration of '__gtk_reserved1' [-Wparentheses]
   void (*__gtk_reserved1);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:75:8: warning: unnecessary parentheses in declaration of '__gtk_reserved2' [-Wparentheses]
   void (*__gtk_reserved2);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:76:8: warning: unnecessary parentheses in declaration of '__gtk_reserved3' [-Wparentheses]
   void (*__gtk_reserved3);
        ^
C:/msys64/mingw64/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:77:8: warning: unnecessary parentheses in declaration of '__gtk_reserved4' [-Wparentheses]
   void (*__gtk_reserved4);
        ^



../../../../vice/src/arch/gtk3/uinetplay.c: In function 'netplay_update_resources':
../../../../vice/src/arch/gtk3/uinetplay.c:119:5: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
     strncpy(p, gtk_entry_get_text(GTK_ENTRY(np_port)), 256);
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~



Making all in hvsc
../../vice/src/debug.c: In function 'debug_maincpu':
../../vice/src/debug.c:453:46: warning: 'sprintf' may write a terminating nul past the end of the destination [-Wformat-overflow=]
         int line_len = sprintf(tempstr, "%s\n", st);
                                              ^
../../vice/src/debug.c:453:24: note: 'sprintf' output between 2 and 129 bytes into a destination of size 128
         int line_len = sprintf(tempstr, "%s\n", st);
                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../vice/src/debug.c: In function 'debug_main65816cpu':
../../vice/src/debug.c:453:46: warning: 'sprintf' may write a terminating nul past the end of the destination [-Wformat-overflow=]
         int line_len = sprintf(tempstr, "%s\n", st);
                                              ^
../../vice/src/debug.c:453:24: note: 'sprintf' output between 2 and 129 bytes into a destination of size 128
         int line_len = sprintf(tempstr, "%s\n", st);
                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../vice/src/debug.c: In function 'debug_drive':
../../vice/src/debug.c:453:46: warning: 'sprintf' may write a terminating nul past the end of the destination [-Wformat-overflow=]
         int line_len = sprintf(tempstr, "%s\n", st);
                                              ^
../../vice/src/debug.c:453:24: note: 'sprintf' output between 2 and 129 bytes into a destination of size 128
         int line_len = sprintf(tempstr, "%s\n", st);
                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../vice/src/util.c: In function 'util_add_extension_maxpath':
../../vice/src/util.c:1431:5: warning: passing argument 1 to restrict-qualified parameter aliases with argument 3 [-Wrestrict]
     sprintf(name, "%s%c%s", name, FSDEV_EXT_SEP_CHR, extension);
     ^~~~~~~


  • compyx (x64, Windows 10/msys2, GCC 8.2.0, SDL2) (r35435)
Making all in sdl
In file included from ../../../../vice/src/arch/sdl/archdep.c:54:
In function 'system_mbstowcs',
    inlined from 'system_mbstowcs_alloc' at ../../../../vice/src/arch/sdl/archdep_win32.c:197:5,
    inlined from 'archdep_default_logger' at ../../../../vice/src/arch/sdl/archdep_win32.c:401:14:
../../../../vice/src/arch/sdl/archdep_win32.c:184:5: warning: 'strncpy' specified bound depends on the length of the source argument [-Wstringop-overflow=]
     strncpy(wcs, mbs, len);
     ^~~~~~~~~~~~~~~~~~~~~~
../../../../vice/src/arch/sdl/archdep_win32.c: In function 'archdep_default_logger':
../../../../vice/src/arch/sdl/archdep_win32.c:197:31: note: length computed here
     system_mbstowcs(wcs, mbs, strlen(mbs) + 1);
                               ^~~~~~~~~~~

clang

  • gpz (x86, Linux/Gentoo, 64bit, clang 5.0.1, GTK3) (3.2 tarball)
CC=clang CXX=clang LD=clang ./configure --enable-native-tools --enable-arch=native --prefix=/usr/games/ --enable-parsid --with-midas --with-resid --with-xaw3d --with-arts --with-x --enable-native-gtk3ui --enable-ethernet --with-pulse --with-sdlsound --enable-fullscreen --enable-cpuhistory --enable-static-ffmpeg

note: here compiling using clang breaks in either ffmpeg or resid 

Making all in gfxoutputdrv
ffmpegdrv.c:126:36: warning: missing field 'next_pts' initializer [-Wmissing-field-initializers]
static OutputStream audio_st = { 0 };
                                   ^
ffmpegdrv.c:139:36: warning: missing field 'next_pts' initializer [-Wmissing-field-initializers]
static OutputStream video_st = { 0 };
                                   ^
ffmpegdrv.c:512:24: warning: missing field 'pts' initializer [-Wmissing-field-initializers]
    AVPacket pkt = { 0 };
                       ^
ffmpegdrv.c:981:28: warning: missing field 'pts' initializer [-Wmissing-field-initializers]
        AVPacket pkt = { 0 };
                           ^

Making all in base
keysetdialog.c:98:44: warning: suggest braces around initialization of subobject [-Wmissing-braces]
static GtkWidget *keyset_buttons[3][3] = { NULL };
                                           ^~~~
                                           {   }                                                                                                                                                     
/usr/lib64/llvm/5/bin/../../../../lib/clang/5.0.1/include/stddef.h:105:16: note: expanded from macro 'NULL'
#  define NULL ((void*)0)
               ^~~~~~~~~~
1 warning generated.
videopalettewidget.c:90:9: warning: unused variable 'index' [-Wunused-variable]
    int index = gtk_combo_box_get_active(combo);
        ^
1 warning generated.
ui.c:427:24: warning: unused variable 'tmp' [-Wunused-variable]
                gchar *tmp = g_filename_from_uri(files[i], NULL, NULL);
                       ^
1 warning generated.
uinetplay_new.c:88:9: warning: unused variable 'state' [-Wunused-variable]
    int state = gtk_switch_get_active(widget);
        ^

Making all in ffmpeg
In file included from info.c:34:
./infocontrib.h:293:8: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
"  Andr<E9> Fachat <fachat@physik.tu-chemnitz.de>\n"
       ^~~~
./infocontrib.h:343:12: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
"  Paul Dub<E9> \n"
           ^~~~
./infocontrib.h:444:8: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
"  Istv<E1>n F<E1>bi<E1>n\n"
       ^~~~   ~~~~  ~~~~
./infocontrib.h:532:11: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
"  Frank K<F6>nig <frank_koenig@t-online.de>\n"
          ^~~~
./infocontrib.h:571:11: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
"  Marko M<E4>kel<E4> <msmakela@gmail.com>\n"
          ^~~~   ~~~~
./infocontrib.h:622:10: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
"  Lasse <D6><F6>rni <loorni@mail.student.oulu.fi>\n"
         ^~~~~~~~
./infocontrib.h:655:14: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
"  Pablo Rold<E1>n <pdroldan@yahoo.com>\n"
             ^~~~
./infocontrib.h:799:19: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
"  Peternel, Per H<E5>kan Sundell and David Horrocks for writing\n"
                  ^~~~
./infocontrib.h:841:25: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
    { "1996-2001", "Andr<E9> Fachat", "Andr<E9> Fachat <fachat@physik.tu-chemnitz.de>" },
                        ^~~~
./infocontrib.h:841:41: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
    { "1996-2001", "Andr<E9> Fachat", "Andr<E9> Fachat <fachat@physik.tu-chemnitz.de>" },
                                           ^~~~
./infocontrib.h:862:29: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
    { "2004-2017", "Paul Dub<E9>", "French", "Paul Dub<E9> " },
                            ^~~~
./infocontrib.h:862:52: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
    { "2004-2017", "Paul Dub<E9>", "French", "Paul Dub<E9> " },
                                                      ^~~~
12 warnings generated.
zfile.c:490:17: warning: logical not is only applied to the left hand side of this comparison [-Wlogical-not-parentheses]
                !strcasecmp(tmp + l - len, search) != 0) {
                ^                                  ~~
zfile.c:490:17: note: add parentheses after the '!' to evaluate the comparison first
                !strcasecmp(tmp + l - len, search) != 0) {
                ^
                 (                                     )                                                                                                                                             
zfile.c:490:17: note: add parentheses around left hand side expression to silence this warning
                !strcasecmp(tmp + l - len, search) != 0) {
                ^
                (                                 )                                                                                                                                                  
1 warning generated.
Retrieved from "http://vice-emu.pokefinder.org/vicewiki/index.php?title=Warnings&oldid=4129"