Merge pull request #19 from duckdb/yl/minor-cleanup

Minor cleanup

src/http_server.cpp

@@ -40,20 +40,23 @@ void HttpServer::UpdateDatabaseInstanceIfRunning(
 void HttpServer::UpdateDatabaseInstance(
     shared_ptr<DatabaseInstance> context_db) {
   const auto current_db = server_instance->LockDatabaseInstance();
-  if (current_db != context_db) {
-    auto watcher_stopped = false;
-    if (server_instance->watcher) {
+  if (current_db == context_db) {
+    return;
+  }
+
+  bool has_watcher = !!server_instance->watcher;
+  if (has_watcher) {
     server_instance->watcher->Stop();
     server_instance->watcher = nullptr;
-      watcher_stopped = true;
   }
+
   server_instance->ddb_instance = context_db;
-    if (watcher_stopped) {
+
+  if (has_watcher) {
     server_instance->watcher = make_uniq<Watcher>(*this);
     server_instance->watcher->Start();
   }
 }
-}
 
 bool HttpServer::IsRunningOnMachine(ClientContext &context) {
   if (Started()) {
@@ -64,11 +67,7 @@ bool HttpServer::IsRunningOnMachine(ClientContext &context) {
   auto local_url = StringUtil::Format("http://localhost:%d", local_port);
 
   httplib::Client client(local_url);
-  auto result = client.Get("/info");
-  if (result) {
-    return true;
-  }
-  return false;
+  return client.Get("/info");
 }
 
 bool HttpServer::Started() {
@@ -204,6 +203,7 @@ void HttpServer::HandleGetLocalEvents(const httplib::Request &req,
         if (event_dispatcher->WaitEvent(&sink)) {
           return true;
         }
+
         sink.done();
         return false;
       });

src/watcher.cpp

@@ -9,7 +9,7 @@
 namespace duckdb {
 namespace ui {
 
-Watcher::Watcher(HttpServer &_server) : server(_server), should_run(false) {}
+Watcher::Watcher(HttpServer &_server) : should_run(false), server(_server) {}
 
 bool WasCatalogUpdated(DatabaseInstance &db, Connection &connection,
                        CatalogState &last_state) {