From 73eb49cf81373b5b561db5f927a4236f7208c117 Mon Sep 17 00:00:00 2001
From: Chengyu Zheng <chengyu.zheng@polimi.it>
Date: Mon, 23 Apr 2018 18:51:10 +0200
Subject: [PATCH] remove output pos <X,Y> syntax

---
 sway/commands/output.c | 30 ++++++++----------------------
 sway/sway.5.txt        |  2 +-
 2 files changed, 9 insertions(+), 23 deletions(-)

diff --git a/sway/commands/output.c b/sway/commands/output.c
index f7e3372ce..74728a9c9 100644
--- a/sway/commands/output.c
+++ b/sway/commands/output.c
@@ -73,29 +73,15 @@ static struct cmd_results *cmd_output_position(struct output_config *output,
 
 	char *end;
 	output->x = strtol(argv[*i], &end, 10);
+	// Format is 1234 4321
+	if (++*i >= argc) {
+		return cmd_results_new(CMD_INVALID, "output",
+			"Missing position argument (y).");
+	}
+	output->y = strtol(argv[*i], &end, 10);
 	if (*end) {
-		// Format is 1234,4321
-		if (*end != ',') {
-			return cmd_results_new(CMD_INVALID, "output",
-				"Invalid position x.");
-		}
-		++end;
-		output->y = strtol(end, &end, 10);
-		if (*end) {
-			return cmd_results_new(CMD_INVALID, "output",
-				"Invalid position y.");
-		}
-	} else {
-		// Format is 1234 4321 (legacy)
-		if (++*i >= argc) {
-			return cmd_results_new(CMD_INVALID, "output",
-				"Missing position argument (y).");
-		}
-		output->y = strtol(argv[*i], &end, 10);
-		if (*end) {
-			return cmd_results_new(CMD_INVALID, "output",
-				"Invalid position y.");
-		}
+		return cmd_results_new(CMD_INVALID, "output",
+			"Invalid position y.");
 	}
 
 	return NULL;
diff --git a/sway/sway.5.txt b/sway/sway.5.txt
index 039753493..1cd267c02 100644
--- a/sway/sway.5.txt
+++ b/sway/sway.5.txt
@@ -374,7 +374,7 @@ The default colors are:
 	+
 	output HDMI-A-1 mode 1920x1080@60Hz
 
-**output** <name> position|pos <X,Y>::
+**output** <name> position|pos <X> <Y>::
 	Configures the specified output to be arranged at the given position.
 
 **output** <name> scale <I>::