Corrected several issues identified during review
This commit is contained in:
parent
86ba01b1d9
commit
7dc68a18bf
|
@ -407,9 +407,6 @@ func (c *client) doCommand(action string, values []string) {
|
||||||
|
|
||||||
func (c *client) doCommandAs(action string, values []string) {
|
func (c *client) doCommandAs(action string, values []string) {
|
||||||
switch action {
|
switch action {
|
||||||
case "C", "CHECK":
|
|
||||||
c.displayConfigValue(values[0])
|
|
||||||
c.DrawMessage()
|
|
||||||
case "ADD", "A":
|
case "ADD", "A":
|
||||||
if len(values) < 2 {
|
if len(values) < 2 {
|
||||||
c.SetMessage(syntaxErrorMessage(action), true)
|
c.SetMessage(syntaxErrorMessage(action), true)
|
||||||
|
@ -502,9 +499,6 @@ func (c *client) doLinkCommandAs(action, target string, values []string) {
|
||||||
}
|
}
|
||||||
|
|
||||||
switch action {
|
switch action {
|
||||||
case "C", "CHECK":
|
|
||||||
c.displayConfigValue(values[0])
|
|
||||||
c.DrawMessage()
|
|
||||||
case "ADD", "A":
|
case "ADD", "A":
|
||||||
bm := make([]string, 0, 5)
|
bm := make([]string, 0, 5)
|
||||||
bm = append(bm, links[num])
|
bm = append(bm, links[num])
|
||||||
|
|
|
@ -94,10 +94,10 @@ func (p *Parser) parseAction() (*Command, error) {
|
||||||
case Value:
|
case Value:
|
||||||
cm.Target = t.val
|
cm.Target = t.val
|
||||||
cm.Type = DOLINK
|
cm.Type = DOLINK
|
||||||
case Word, Action:
|
case Word:
|
||||||
cm.Value = append(cm.Value, t.val)
|
cm.Value = append(cm.Value, t.val)
|
||||||
cm.Type = DO
|
cm.Type = DO
|
||||||
case Whitespace:
|
case Action, Whitespace:
|
||||||
return nil, fmt.Errorf("Found %q (%d), expected value", t.val, t.kind)
|
return nil, fmt.Errorf("Found %q (%d), expected value", t.val, t.kind)
|
||||||
}
|
}
|
||||||
t = p.scan()
|
t = p.scan()
|
||||||
|
|
2
help.go
2
help.go
|
@ -1,6 +1,6 @@
|
||||||
package main
|
package main
|
||||||
|
|
||||||
// ERRS maps commands to their help text
|
// ERRS maps commands to their syntax error message
|
||||||
var ERRS = map[string]string{
|
var ERRS = map[string]string{
|
||||||
"A": "`a [target] [name...]`",
|
"A": "`a [target] [name...]`",
|
||||||
"ADD": "`add [target] [name...]`",
|
"ADD": "`add [target] [name...]`",
|
||||||
|
|
Loading…
Reference in New Issue