From 5115acc0d3478cd323a5129ae2c12b5b8ba735d0 Mon Sep 17 00:00:00 2001
From: Akash Yadav <itsaky01@gmail.com>
Date: Sun, 26 Jun 2022 09:30:39 +0530
Subject: [PATCH] Removed debug method calls

---
 app/src/main/AndroidManifest.xml                 |  2 --
 .../com/itsaky/androidide/EditorActivity.java    |  1 -
 .../androidide/lsp/IDELanguageClientImpl.java    |  2 --
 .../com/itsaky/lsp/java/JavaLanguageServer.java  | 16 ----------------
 4 files changed, 21 deletions(-)

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 880d10f33..a42b8a619 100755
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -26,8 +26,6 @@
         android:supportsRtl="true"
         android:theme="@style/AppTheme"
         tools:ignore="UnusedAttribute,MediaCapabilities">
-        <profileable android:shell="true" />
-
         <activity
             android:name=".CrashHandlerActivity"
             android:exported="true"
diff --git a/app/src/main/java/com/itsaky/androidide/EditorActivity.java b/app/src/main/java/com/itsaky/androidide/EditorActivity.java
index 4121150fb..286beafa5 100755
--- a/app/src/main/java/com/itsaky/androidide/EditorActivity.java
+++ b/app/src/main/java/com/itsaky/androidide/EditorActivity.java
@@ -241,7 +241,6 @@ public class EditorActivity extends StudioActivity
 
   @Override
   public boolean onPrepareOptionsMenu(Menu menu) {
-    LOG.debug("Prepare options menu");
     ensureToolbarMenu(menu);
     return true;
   }
diff --git a/app/src/main/java/com/itsaky/androidide/lsp/IDELanguageClientImpl.java b/app/src/main/java/com/itsaky/androidide/lsp/IDELanguageClientImpl.java
index 0ffa9c588..a903e01c1 100755
--- a/app/src/main/java/com/itsaky/androidide/lsp/IDELanguageClientImpl.java
+++ b/app/src/main/java/com/itsaky/androidide/lsp/IDELanguageClientImpl.java
@@ -160,7 +160,6 @@ public class IDELanguageClientImpl implements ILanguageClient {
     File file = result.getFile().toFile();
     if (!file.exists() || !file.isFile()) return;
 
-    Trace.beginSection("publishDiagnostics");
     final var editorView = activity().getEditorForFile(file);
     if (editorView != null) {
       final var editor = editorView.getEditor();
@@ -180,7 +179,6 @@ public class IDELanguageClientImpl implements ILanguageClient {
 
     diagnostics.put(file, result.getDiagnostics());
     activity().setDiagnosticsAdapter(newDiagnosticsAdapter());
-    Trace.endSection();
   }
 
   @Nullable
diff --git a/lsp/java/src/main/java/com/itsaky/lsp/java/JavaLanguageServer.java b/lsp/java/src/main/java/com/itsaky/lsp/java/JavaLanguageServer.java
index 922f375bf..55ef5ca5f 100644
--- a/lsp/java/src/main/java/com/itsaky/lsp/java/JavaLanguageServer.java
+++ b/lsp/java/src/main/java/com/itsaky/lsp/java/JavaLanguageServer.java
@@ -17,8 +17,6 @@
 
 package com.itsaky.lsp.java;
 
-import android.os.Looper;
-
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.annotation.RestrictTo;
@@ -92,12 +90,7 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
     applySettings(getSettings());
   }
 
-  private void logIfMainThread(String methodName) {
-    LOG.debug(methodName, ": Is main thread?", Looper.myLooper() == Looper.getMainLooper());
-  }
-
   private void analyzeSelected() {
-    logIfMainThread("analyzeSelected");
     if (this.selectedFile == null) {
       return;
     }
@@ -136,7 +129,6 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
 
   @NonNull
   private JavaCompilerService createCompiler() {
-    logIfMainThread("createCompiler");
     return new JavaCompilerService(configuration.getClassPaths(), Collections.emptySet());
   }
 
@@ -205,7 +197,6 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
 
   @Override
   public void configurationChanged(Object newConfiguration) {
-    logIfMainThread("configurationChanged");
     if (!(newConfiguration instanceof JavaServerConfiguration)) {
       LOG.error("Invalid configuration passed to server.", newConfiguration);
       LOG.error("Configuration change event will be ignored.");
@@ -228,7 +219,6 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
   @NonNull
   @Override
   public ICompletionProvider getCompletionProvider() {
-    logIfMainThread("completionProvider");
     if (!settings.completionsEnabled()) {
       return new NoCompletionsProvider();
     }
@@ -270,7 +260,6 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
   @NonNull
   @Override
   public SignatureHelp signatureHelp(@NonNull SignatureHelpParams params) {
-    logIfMainThread("signatureHelp");
     if (!settings.signatureHelpEnabled()) {
       return new SignatureHelp(Collections.emptyList(), -1, -1);
     }
@@ -281,7 +270,6 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
   @NonNull
   @Override
   public List<DiagnosticItem> analyze(@NonNull Path file) {
-    logIfMainThread("analyze file:" + file);
     if (!settings.codeAnalysisEnabled()) {
       return Collections.emptyList();
     }
@@ -313,7 +301,6 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
 
   @Override
   public void onFileOpened(DocumentOpenEvent event) {
-    logIfMainThread("onFileOpened");
     onFileSelected(event.getOpenedFile());
     ensureAnalyzeTimerStarted();
     FileStore.open(event);
@@ -321,7 +308,6 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
 
   @Override
   public void onContentChange(@NonNull DocumentChangeEvent event) {
-    logIfMainThread("onContentChanged");
     // If a file's content is changed, it is definitely visible to user.
     onFileSelected(event.getChangedFile());
     ensureAnalyzeTimerStarted();
@@ -335,7 +321,6 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
 
   @Override
   public void onFileClosed(DocumentCloseEvent event) {
-    logIfMainThread("onFileClosed");
     FileStore.close(event);
   }
 
@@ -345,7 +330,6 @@ public class JavaLanguageServer implements ILanguageServer, IDocumentHandler {
   }
 
   private void ensureAnalyzeTimerStarted() {
-    logIfMainThread("ensureAnalyzeTimerStarted");
     if (!this.analyzeTimer.isStarted()) {
       this.analyzeTimer.start();
     } else {