monotone-commits-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Monotone-commits-diffs] net.venge.monotone.colored-diff: 96c709bbdd68e


From: code
Subject: [Monotone-commits-diffs] net.venge.monotone.colored-diff: 96c709bbdd68e6b620813e6221489509b93bd93c
Date: Sun, 16 Jan 2011 12:58:15 GMT

revision:            96c709bbdd68e6b620813e6221489509b93bd93c
date:                2011-01-15T23:01:40
author:              Richard Hopkins
branch:              net.venge.monotone.colored-diff
changelog:
Changed default get_output_color Lua hook to use a table

This is a readability improvement over the old if/elseif
style for returning the color.

manifest:
format_version "1"

new_manifest [b5ead76beb70bbbe045bf6c2a4976f638b873205]

old_revision [f6f7c9b8d1c63bebcba69c09111506145d4ee4f6]

patch "std_hooks.lua"
 from [3b2d4491eb3582637f4479f74cffb15cf0ef702f]
   to [3497e715f6d10ab3f3224a4dfa1448dd918f812d]
============================================================
--- std_hooks.lua	3b2d4491eb3582637f4479f74cffb15cf0ef702f
+++ std_hooks.lua	3497e715f6d10ab3f3224a4dfa1448dd918f812d
@@ -1538,27 +1538,28 @@ function get_output_color(purpose)
 	-- valid return values are
 	-- "" for monotone to use the default output
 	-- black, white, red, green, blue, magenta, yellow, cyan, bold, reset
-	if purpose == "normal" then
+
+	local color_table = 
+	{
+		normal = "",
+		reset = "reset",
+		
+		diff_add = "green",
+		diff_change = "magenta",
+		diff_comment = "cyan",
+		diff_delete = "red",
+		diff_encloser = "bold",
+		diff_separator = "bold",
+		
+		log_revision = "bold",
+		rev_header = "bold"
+	}
+
+	local chosen_color = color_table[purpose]
+	
+	if chosen_color == nil then
 		return ""
-	elseif purpose == "reset" then
-		return "reset"
-	elseif purpose == "diff_add" then
-		return "green"
-	elseif purpose == "diff_change" then
-		return "magenta"
-	elseif purpose == "diff_comment" then
-		return "cyan"
-	elseif purpose == "diff_delete" then
-		return "red"
-	elseif purpose == "diff_encloser" then
-		return "bold"
-	elseif purpose == "diff_separator" then
-		return "bold"
-	elseif purpose == "log_revision" then
-		return "bold"
-	elseif purpose == "rev_header" then
-		return "bold"
 	else
-		return ""
+		return chosen_color
 	end
 end

reply via email to

[Prev in Thread] Current Thread [Next in Thread]