From 91f9f355b2e334214e38e1827c624cdcc23c5130 Mon Sep 17 00:00:00 2001 From: obscuren Date: Fri, 20 Mar 2015 12:17:57 +0100 Subject: [PATCH 1/3] test --- cmd/ethereum/main.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/ethereum/main.go b/cmd/ethereum/main.go index 38c04ec9d..1a662c756 100644 --- a/cmd/ethereum/main.go +++ b/cmd/ethereum/main.go @@ -278,7 +278,7 @@ func accountCreate(ctx *cli.Context) { if err != nil { utils.Fatalf("Could not create the account: %v", err) } - fmt.Printf("Address: %#x\n", acct.Address) + fmt.Printf("Address: %x\n", acct.Address) } func importchain(ctx *cli.Context) { From 4f5b362bda862b0d77583491d5e04193148a9349 Mon Sep 17 00:00:00 2001 From: obscuren Date: Fri, 20 Mar 2015 12:52:03 +0100 Subject: [PATCH 2/3] %#x => %x --- cmd/ethereum/main.go | 2 +- p2p/discover/node.go | 4 ++-- p2p/message.go | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/cmd/ethereum/main.go b/cmd/ethereum/main.go index 1a662c756..87f588de8 100644 --- a/cmd/ethereum/main.go +++ b/cmd/ethereum/main.go @@ -251,7 +251,7 @@ func accountList(ctx *cli.Context) { utils.Fatalf("Could not list accounts: %v", err) } for _, acct := range accts { - fmt.Printf("Address: %#x\n", acct) + fmt.Printf("Address: %x\n", acct) } } diff --git a/p2p/discover/node.go b/p2p/discover/node.go index de2588258..e1130e0b5 100644 --- a/p2p/discover/node.go +++ b/p2p/discover/node.go @@ -143,12 +143,12 @@ type NodeID [nodeIDBits / 8]byte // NodeID prints as a long hexadecimal number. func (n NodeID) String() string { - return fmt.Sprintf("%#x", n[:]) + return fmt.Sprintf("%x", n[:]) } // The Go syntax representation of a NodeID is a call to HexID. func (n NodeID) GoString() string { - return fmt.Sprintf("discover.HexID(\"%#x\")", n[:]) + return fmt.Sprintf("discover.HexID(\"%x\")", n[:]) } // HexID converts a hex string to a NodeID. diff --git a/p2p/message.go b/p2p/message.go index 14e4404c9..31cf5901f 100644 --- a/p2p/message.go +++ b/p2p/message.go @@ -40,7 +40,7 @@ func NewMsg(code uint64, params ...interface{}) Msg { // For the decoding rules, please see package rlp. func (msg Msg) Decode(val interface{}) error { if err := rlp.Decode(msg.Payload, val); err != nil { - return newPeerError(errInvalidMsg, "(code %#x) (size %d) %v", msg.Code, msg.Size, err) + return newPeerError(errInvalidMsg, "(code %x) (size %d) %v", msg.Code, msg.Size, err) } return nil } From 9edb9a21bce97594928f8660e8e32df2cb25b74d Mon Sep 17 00:00:00 2001 From: Taylor Gerring Date: Sat, 21 Mar 2015 07:26:44 +0100 Subject: [PATCH 3/3] Add json LogSystem #538 --- logger/log.go | 2 ++ logger/logsystem.go | 24 ++++++++++++++++++++++++ logger/sys.go | 3 +-- 3 files changed, 27 insertions(+), 2 deletions(-) diff --git a/logger/log.go b/logger/log.go index 2dbec86de..fab004d0a 100644 --- a/logger/log.go +++ b/logger/log.go @@ -30,6 +30,8 @@ func New(datadir string, logFile string, logLevel int, logFormat string) LogSyst switch logFormat { case "raw": sys = NewRawLogSystem(writer, 0, LogLevel(logLevel)) + case "json": + sys = NewJsonLogSystem(writer, 0, LogLevel(logLevel)) default: sys = NewStdLogSystem(writer, log.LstdFlags, LogLevel(logLevel)) } diff --git a/logger/logsystem.go b/logger/logsystem.go index 8458b938f..1318a9f96 100644 --- a/logger/logsystem.go +++ b/logger/logsystem.go @@ -61,3 +61,27 @@ func (t *rawLogSystem) SetLogLevel(i LogLevel) { func (t *rawLogSystem) GetLogLevel() LogLevel { return LogLevel(atomic.LoadUint32(&t.level)) } + +// NewRawLogSystem creates a LogSystem that prints to the given writer without +// adding extra information. Suitable for preformatted output +func NewJsonLogSystem(writer io.Writer, flags int, level LogLevel) LogSystem { + logger := log.New(writer, "", 0) + return &jsonLogSystem{logger, uint32(level)} +} + +type jsonLogSystem struct { + logger *log.Logger + level uint32 +} + +func (t *jsonLogSystem) LogPrint(level LogLevel, msg string) { + t.logger.Print(msg) +} + +func (t *jsonLogSystem) SetLogLevel(i LogLevel) { + atomic.StoreUint32(&t.level, uint32(i)) +} + +func (t *jsonLogSystem) GetLogLevel() LogLevel { + return LogLevel(atomic.LoadUint32(&t.level)) +} diff --git a/logger/sys.go b/logger/sys.go index bd826b587..db4251a52 100644 --- a/logger/sys.go +++ b/logger/sys.go @@ -76,8 +76,7 @@ func dispatchLoop() { func sysLoop(sys LogSystem, in <-chan message, wg *sync.WaitGroup) { for msg := range in { switch sys.(type) { - case *rawLogSystem: - // This is a semantic hack since rawLogSystem has little to do with JsonLevel + case *jsonLogSystem: if msg.level == JsonLevel { sys.LogPrint(msg.level, msg.msg) }