edit: De-duplicate code in edit()'s main switch

Signed-off-by: Markus Armbruster <armbru@pond.sub.org>

diff --git a/src/lib/commands/edit.c b/src/lib/commands/edit.c
index 6f85648530d8bedf74d1d7f30128639a045f212e..7258bd0cafeb17d6120a77a58d7e9c8d003b7306 100644 (file)
--- a/src/lib/commands/edit.c
+++ b/src/lib/commands/edit.c
@@ -157,10 +157,10 @@ edit(void)
        } else
            return RET_OK;
 
        } else
            return RET_OK;
 

+       if (!check_obj_ok(&item.gen))
+           return RET_FAIL;

        switch (ewhat) {
        case 'c':
        switch (ewhat) {
        case 'c':

-           if (!check_obj_ok(&item.gen))
-               return RET_FAIL;

            /*
             * edit_nat() may update the edited country by sending it
             * bulletins.  Writing back item.nat would trigger a seqno
            /*
             * edit_nat() may update the edited country by sending it
             * bulletins.  Writing back item.nat would trigger a seqno


@@ -172,46 +172,26 @@ edit(void)
            if (!putnat(np))
                return RET_FAIL;
            item.nat = *np;
            if (!putnat(np))
                return RET_FAIL;
            item.nat = *np;

-           break;
+           continue;

        case 'l':
        case 'l':

-           if (!check_sect_ok(&item.sect))
-               return RET_FAIL;

            ret = edit_sect(&item.sect, key, ptr);
            ret = edit_sect(&item.sect, key, ptr);

-           if (ret != RET_OK)
-               return ret;
-           if (!putsect(&item.sect))
-               return RET_FAIL;

            break;
        case 's':
            break;
        case 's':

-           if (!check_ship_ok(&item.ship))
-               return RET_FAIL;

            ret = edit_ship(&item.ship, key, ptr);
            ret = edit_ship(&item.ship, key, ptr);

-           if (ret != RET_OK)
-               return ret;
-           if (!putship(item.ship.shp_uid, &item.ship))
-               return RET_FAIL;

            break;
        case 'u':
            break;
        case 'u':

-           if (!check_land_ok(&item.land))
-               return RET_FAIL;

            ret = edit_land(&item.land, key, ptr);
            ret = edit_land(&item.land, key, ptr);

-           if (ret != RET_OK)
-               return ret;
-           if (!putland(item.land.lnd_uid, &item.land))
-               return RET_FAIL;

            break;
        case 'p':
            break;
        case 'p':

-           if (!check_plane_ok(&item.plane))
-               return RET_FAIL;

            ret = edit_plane(&item.plane, key, ptr);
            ret = edit_plane(&item.plane, key, ptr);

-           if (ret != RET_OK)
-               return ret;
-           if (!putplane(item.plane.pln_uid, &item.plane))
-               return RET_FAIL;

            break;
        default:
            CANT_REACH();
        }
            break;
        default:
            CANT_REACH();
        }

+       if (ret != RET_OK)
+           return ret;
+       if (!put_empobj(item.gen.ef_type, item.gen.uid, &item.gen))
+           return RET_FAIL;

     }
 }
 
     }
 }