From 5646b997786c9d11b9e1fda98b855bef6320da54 Mon Sep 17 00:00:00 2001
From: Alexander von Gluck IV <kallisti5@unixzen.com>
Date: Sun, 11 Oct 2020 09:16:00 -0500
Subject: [PATCH] input/MouseSettings: Fix return to NULL per clang

* This looks wrong, but is functionally equal.
* Supposed to return a MouseSettings*, but returned
  a B_OK (0) on a seemingly error result (NULL).

Change-Id: Id6c50369790b90e38fd6ec037c466c5fb314edf5
Reviewed-on: https://review.haiku-os.org/c/haiku/+/3303
Reviewed-by: Adrien Destugues <pulkomandy@gmail.com>
Reviewed-by: waddlesplash <waddlesplash@gmail.com>
---
 src/servers/input/MouseSettings.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/servers/input/MouseSettings.cpp b/src/servers/input/MouseSettings.cpp
index 1699845de9..d348cd75ab 100644
--- a/src/servers/input/MouseSettings.cpp
+++ b/src/servers/input/MouseSettings.cpp
@@ -422,12 +422,12 @@ MultipleMouseSettings::AddMouseSettings(BString mouse_name)
 
 	MouseSettings* settings = new (std::nothrow) MouseSettings();
 
-	if(settings !=NULL) {
+	if(settings != NULL) {
 		fMouseSettingsObject.insert(std::pair<BString, MouseSettings*>
 			(mouse_name, settings));
 		return settings;
 	}
-	return B_OK;
+	return NULL;
 }