Minor refactoring, add gover managed versioning

cmd/wol/alias.go

@@ -164,5 +164,3 @@ func (a *Aliases) Close() error {
 
 	return a.db.Close()
 }
-
-////////////////////////////////////////////////////////////////////////////////

cmd/wol/alias_test.go

@@ -213,5 +213,3 @@ func (suite *AliasDBTests) TestGetAlias() {
 func TestRunAllSuites(t *testing.T) {
 	suite.Run(t, new(AliasDBTests))
 }
-
-////////////////////////////////////////////////////////////////////////////////

cmd/wol/usage.go

@@ -99,5 +99,3 @@ func isValidCommand(s string) bool {
 	}
 	return false
 }
-
-////////////////////////////////////////////////////////////////////////////////

cmd/wol/version_gen.go

@@ -0,0 +1,13 @@
+package main
+
+// WARNING: Auto generated version file. Do not edit this file by hand.
+// WARNING: go get github.com/sabhiram/gover to manage this file.
+// Version: 1.0.3
+
+const (
+    Major = 1
+    Minor = 0
+    Patch = 3
+
+    Version = "1.0.3"
+)

cmd/wol/wol.go

@@ -167,24 +167,24 @@ func main() {
 	parser := flags.NewParser(&Options, flags.Default & ^flags.HelpFlag)
 	args, err = parser.Parse()
 
-	exitCode := 0
+	ec := 0
 	switch {
 
 	// Parse Error, print usage
 	case err != nil:
-		exitCode = printUsageGetExitCode("", 1)
+		ec = printUsageGetExitCode("", 1)
 
 	// No arguments, or help requested, print usage
 	case len(os.Args) == 1 || Options.Help:
-		exitCode = printUsageGetExitCode("", 0)
+		ec = printUsageGetExitCode("", 0)
 
 	// "--version" requested
 	case Options.Version:
 		fmt.Printf("%s\n", Version)
 
 	// Make sure we are being asked to run a something
 	case len(args) == 0:
-		exitCode = printUsageGetExitCode("No command specified, see usage:\n", 1)
+		ec = printUsageGetExitCode("No command specified, see usage:\n", 1)
 
 	// All other cases go here
 	case true:
@@ -197,11 +197,9 @@ func main() {
 
 		if err != nil {
 			fmt.Printf("%s\n", err.Error())
-			exitCode = 1
+			ec = 1
 		}
 
 	}
-	os.Exit(exitCode)
+	os.Exit(ec)
 }
-
-////////////////////////////////////////////////////////////////////////////////

magic_packet.go

@@ -136,14 +136,12 @@ func SendMagicPacket(macAddr, bcastAddr, iface string) error {
 	defer connection.Close()
 
 	// Write the bytes of the MagicPacket to the connection
-	bytesWritten, err := connection.Write(buf.Bytes())
+	n, err := connection.Write(buf.Bytes())
 	if err != nil {
 		return err
-	} else if bytesWritten != 102 {
-		fmt.Printf("Warning: %d bytes written, %d expected!\n", bytesWritten, 102)
+	} else if n != 102 {
+		fmt.Printf("Warning: %d bytes written, %d expected!\n", n, 102)
 	}
 
 	return nil
 }
-
-////////////////////////////////////////////////////////////////////////////////

magic_packet_test.go

@@ -78,5 +78,3 @@ func TestGetIpFromInterfaceNegative(t *testing.T) {
 		assert.NotNil(t, err)
 	}
 }
-
-////////////////////////////////////////////////////////////////////////////////