[PATCH wayland v3 1/7] socket-test: Fix some comment typos

Bryce Harrington bryce at osg.samsung.com
Mon Dec 7 22:49:13 PST 2015


Signed-off-by: Bryce Harrington <bryce at osg.samsung.com>
---
 tests/socket-test.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/tests/socket-test.c b/tests/socket-test.c
index c53f972..2e336fd 100644
--- a/tests/socket-test.c
+++ b/tests/socket-test.c
@@ -107,11 +107,11 @@ TEST(add_existing_socket)
 	ret = wl_display_add_socket(d, name);
 	assert(ret == 0);
 
-	/* this on should fail */
+	/* this one should fail */
 	ret = wl_display_add_socket(d, name);
 	assert(ret < 0);
 
-	/* the original socket should still exists,
+	/* the original socket should still exist.
 	 * this was a bug introduced in e2c0d47b0c77f18cd90e9c6eabb358c4d89681c8 */
 	len = snprintf(path, sizeof example_sockaddr_un.sun_path, "%s/%s",
 		       xdg_runtime_dir, name);
@@ -120,7 +120,7 @@ TEST(add_existing_socket)
 
 	assert(access(path, F_OK) != -1);
 
-	/* still should exists the original socket */
+	/* the original socket should still exist */
 	ret = wl_display_add_socket(d, name);
 	assert(ret < 0);
 
@@ -129,7 +129,9 @@ TEST(add_existing_socket)
 
 TEST(add_socket_auto)
 {
-	/* the number of auto sockets is currently 32 */
+	/* the number of auto sockets is currently 32,
+	 * set in wayland-server.c.
+	 */
 	const int MAX_SOCKETS = 32;
 
 	char path[sizeof example_sockaddr_un.sun_path];
@@ -153,7 +155,7 @@ TEST(add_socket_auto)
 		assert(len < sizeof example_sockaddr_un.sun_path
 		       && "Bug in test. Path too long");
 
-		/* was the socket? */
+		/* was the socket created correctly? */
 		assert(access(path, F_OK) != -1);
 
 		/* is the name sequential? */
-- 
1.9.1



More information about the wayland-devel mailing list