diff options
| author | Stefan Kangas <stefankangas@gmail.com> | 2025-01-26 20:04:07 +0100 |
|---|---|---|
| committer | Stefan Kangas <stefankangas@gmail.com> | 2025-01-26 20:09:08 +0100 |
| commit | 6e2e7265a04f63f482db7fbdfd8e2519d8bfe03e (patch) | |
| tree | 9481b02cc364816c4bb47308685debd17560fc7f /src | |
| parent | b07c12ce59c96db753a1f8f3a829f9ab5514e35f (diff) | |
Prefer static switch-case checking in pdumper
* src/pdumper.c (dump_fwd, dump_symbol, dump_drain_cold_data)
(dump_do_fixup, dump_anonymous_allocate_w32)
(dump_anonymous_allocate_posix, dump_map_file_w32, dump_map_file_posix)
(dump_do_emacs_relocation): Remove default clauses to allow static
checking using -Wswitch.
Diffstat (limited to 'src')
| -rw-r--r-- | src/pdumper.c | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/src/pdumper.c b/src/pdumper.c index e7e6e2c392c..77d4407935e 100644 --- a/src/pdumper.c +++ b/src/pdumper.c @@ -2390,8 +2390,6 @@ dump_fwd (struct dump_context *ctx, lispfwd fwd) case Lisp_Fwd_Kboard_Obj: offset = dump_fwd_kboard_obj (ctx, p); break; - default: - emacs_abort (); } return offset; @@ -2523,8 +2521,6 @@ dump_symbol (struct dump_context *ctx, case SYMBOL_FORWARDED: dump_field_fixup_later (ctx, &out, symbol, &symbol->u.s.val.fwd); break; - default: - emacs_abort (); } dump_field_lv (ctx, &out, symbol, &symbol->u.s.function, WEIGHT_NORMAL); dump_field_lv (ctx, &out, symbol, &symbol->u.s.plist, WEIGHT_NORMAL); @@ -3603,8 +3599,6 @@ dump_drain_cold_data (struct dump_context *ctx) dump_cold_native_subr (ctx, data); break; #endif - default: - emacs_abort (); } } @@ -4069,8 +4063,6 @@ dump_do_fixup (struct dump_context *ctx, do_write = false; break; } - default: - emacs_abort (); } if (do_write) dump_write (ctx, &dump_value, sizeof (dump_value)); @@ -4529,8 +4521,6 @@ dump_anonymous_allocate_w32 (void *base, mem_type = MEM_COMMIT; mem_prot = PAGE_READWRITE; break; - default: - emacs_abort (); } ret = VirtualAlloc (base, size, mem_type, mem_prot); @@ -4569,8 +4559,6 @@ dump_anonymous_allocate_posix (void *base, case DUMP_MEMORY_ACCESS_READWRITE: mem_prot = PROT_READ | PROT_WRITE; break; - default: - emacs_abort (); } int mem_flags = MAP_PRIVATE | MAP_ANONYMOUS; @@ -4663,7 +4651,6 @@ dump_map_file_w32 (void *base, int fd, off_t offset, size_t size, case DUMP_MEMORY_ACCESS_READWRITE: protect = PAGE_WRITECOPY; /* for Windows 9X */ break; - default: case DUMP_MEMORY_ACCESS_NONE: case DUMP_MEMORY_ACCESS_READ: protect = PAGE_READONLY; @@ -4691,8 +4678,6 @@ dump_map_file_w32 (void *base, int fd, off_t offset, size_t size, case DUMP_MEMORY_ACCESS_READWRITE: map_access = FILE_MAP_COPY; break; - default: - emacs_abort (); } ret = MapViewOfFileEx (section, @@ -4735,8 +4720,6 @@ dump_map_file_posix (void *base, int fd, off_t offset, size_t size, mem_prot = PROT_READ | PROT_WRITE; mem_flags = MAP_PRIVATE; break; - default: - emacs_abort (); } if (base) @@ -5603,8 +5586,6 @@ dump_do_emacs_relocation (const uintptr_t dump_base, memcpy (emacs_ptr_at (reloc.emacs_offset), &lv, sizeof (lv)); break; } - default: - fatal ("unrecognied relocation type %d", (int) reloc.type); } } |
