cmd/evm: fix incorrect default value for trace.nomemory and trace.noreturndata

[lightclient]

These should default to false since they're later inverted:

go-ethereum/cmd/evm/internal/t8ntool/transition.go

Lines 109 to 114 in ac23e91

	logConfig := &vm.LogConfig{
	DisableStack: ctx.Bool(TraceDisableStackFlag.Name),
	EnableMemory: !ctx.Bool(TraceDisableMemoryFlag.Name),
	EnableReturnData: !ctx.Bool(TraceDisableReturnDataFlag.Name),
	Debug: true,
	}

[holiman]

BoolTFlag

BoolTFlag is a flag with type bool that is true by default

As opposed to a BoolFlag, which is false by default. So right now, we disable memory output by default. The change you propose would make disable false by default, meaning it would be by default enabled. Which is not what we want, because in most cases memory output is too large to work with.

[simonshampoo]

so how do we enable memory output