cmd: use local flags for dump, omit empty code/storage

cmd/geth/chaincmd.go

@@ -165,7 +165,7 @@ Remove blockchain and state databases`,
 			utils.IterativeOutputFlag,
 			utils.ExcludeCodeFlag,
 			utils.ExcludeStorageFlag,
-			utils.IncludeMissingPreimagesFlag,
+			utils.IncludeIncompletesFlag,
 		},
 		Category: "BLOCKCHAIN COMMANDS",
 		Description: `
@@ -525,10 +525,10 @@ func dump(ctx *cli.Context) error {
 			if err != nil {
 				utils.Fatalf("could not create new state: %v", err)
 			}
-			excludeCode := ctx.GlobalBool(utils.ExcludeCodeFlag.Name)
+			excludeCode := ctx.Bool(utils.ExcludeCodeFlag.Name)
-			excludeStorage := ctx.GlobalBool(utils.ExcludeStorageFlag.Name)
+			excludeStorage := ctx.Bool(utils.ExcludeStorageFlag.Name)
-			includeMissing := ctx.GlobalBool(utils.IncludeMissingPreimagesFlag.Name)
+			includeMissing := ctx.Bool(utils.IncludeIncompletesFlag.Name)
-			if ctx.GlobalBool(utils.IterativeOutputFlag.Name) {
+			if ctx.Bool(utils.IterativeOutputFlag.Name) {
 				state.IterativeDump(excludeCode, excludeStorage, !includeMissing, json.NewEncoder(os.Stdout))
 			} else {
 				if includeMissing {

cmd/geth/main.go

@@ -188,13 +188,6 @@ var (
 		utils.MetricsInfluxDBPasswordFlag,
 		utils.MetricsInfluxDBTagsFlag,
 	}
-
-	dumpFlags = []cli.Flag{
-		utils.IterativeOutputFlag,
-		utils.ExcludeCodeFlag,
-		utils.ExcludeStorageFlag,
-		utils.IncludeMissingPreimagesFlag,
-	}
 )
 
 func init() {
@@ -238,7 +231,6 @@ func init() {
 	app.Flags = append(app.Flags, debug.Flags...)
 	app.Flags = append(app.Flags, whisperFlags...)
 	app.Flags = append(app.Flags, metricsFlags...)
-	app.Flags = append(app.Flags, dumpFlags...)
 
 	app.Before = func(ctx *cli.Context) error {
 		logdir := ""

cmd/utils/flags.go

@@ -197,20 +197,20 @@ var (
 		Usage: "List of trusted ULC servers",
 	}
 	IterativeOutputFlag = cli.BoolFlag{
-		Name:  "dump.iterative",
+		Name:  "iterative",
-		Usage: "Print streaming JSON iteratively as json objects, delimited by newlines",
+		Usage: "Print streaming JSON iteratively, delimited by newlines",
 	}
 	ExcludeStorageFlag = cli.BoolFlag{
-		Name:  "dump.nostorage",
+		Name:  "nostorage",
-		Usage: "When set, exclude storage entries (saves db lookups)",
+		Usage: "Exclude storage entries (save db lookups)",
 	}
-	IncludeMissingPreimagesFlag = cli.BoolFlag{
+	IncludeIncompletesFlag = cli.BoolFlag{
-		Name:  "dump.includeincomplete",
+		Name:  "incompletes",
-		Usage: "When set, include also those we do not have address of (missing preimage)",
+		Usage: "Include accounts for which we don't have the address (missing preimage)",
 	}
 	ExcludeCodeFlag = cli.BoolFlag{
-		Name:  "dump.nocode",
+		Name:  "nocode",
-		Usage: "When set, exclude contract code (saves db lookups)",
+		Usage: "Exclude contract code (save db lookups)",
 	}
 	defaultSyncMode = eth.DefaultConfig.SyncMode
 	SyncModeFlag    = TextMarshalerFlag{

core/state/dump.go

@@ -33,8 +33,8 @@ type DumpAccount struct {
 	Nonce     uint64                 `json:"nonce"`
 	Root      string                 `json:"root"`
 	CodeHash  string                 `json:"codeHash"`
-	Code      string                 `json:"code"`
+	Code      string                 `json:"code,omitempty"`
-	Storage   map[common.Hash]string `json:"storage"`
+	Storage   map[common.Hash]string `json:"storage,omitempty"`
 	Address   *common.Address        `json:"address,omitempty"` // Address only present in iterative (line-by-line) mode
 	SecureKey hexutil.Bytes          `json:"key,omitempty"`     // If we don't have address, we can output the key