Skip to content

Commit

Permalink
HackStudio: Remove ProcessStateWidget
Browse files Browse the repository at this point in the history
This was a little banner that would pop up when running something in
the "Build" tab. It didn't look very good and it was mostly distracting
how it would pop in and out of the window.
  • Loading branch information
awesomekling committed Oct 29, 2020
1 parent 18739ba commit 4e0ab1d
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 167 deletions.
1 change: 0 additions & 1 deletion DevTools/HackStudio/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ set(SOURCES
HackStudioWidget.cpp
LanguageClient.cpp
Locator.cpp
ProcessStateWidget.cpp
Project.cpp
ProjectFile.cpp
TerminalWrapper.cpp
Expand Down
102 changes: 0 additions & 102 deletions DevTools/HackStudio/ProcessStateWidget.cpp

This file was deleted.

55 changes: 0 additions & 55 deletions DevTools/HackStudio/ProcessStateWidget.h

This file was deleted.

6 changes: 0 additions & 6 deletions DevTools/HackStudio/TerminalWrapper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@
*/

#include "TerminalWrapper.h"
#include "ProcessStateWidget.h"
#include <AK/String.h>
#include <LibCore/ConfigFile.h>
#include <LibGUI/BoxLayout.h>
Expand Down Expand Up @@ -82,7 +81,6 @@ void TerminalWrapper::run_command(const String& command)
} else if (WIFSIGNALED(wstatus)) {
m_terminal_widget->inject_string(String::formatted("\033[34;1m(Command signaled with {}!)\033[0m\n", strsignal(WTERMSIG(wstatus))));
}
m_process_state_widget->set_tty_fd(-1);
m_pid = -1;

if (on_command_exit)
Expand Down Expand Up @@ -156,9 +154,6 @@ void TerminalWrapper::run_command(const String& command)
}
ASSERT_NOT_REACHED();
}

// Parent process, cont'd.
m_process_state_widget->set_tty_fd(ptm_fd);
}

void TerminalWrapper::kill_running_command()
Expand All @@ -176,7 +171,6 @@ TerminalWrapper::TerminalWrapper(bool user_spawned)

RefPtr<Core::ConfigFile> config = Core::ConfigFile::get_for_app("Terminal");
m_terminal_widget = add<TerminalWidget>(-1, false, config);
m_process_state_widget = add<ProcessStateWidget>();

if (user_spawned)
run_command("Shell");
Expand Down
3 changes: 0 additions & 3 deletions DevTools/HackStudio/TerminalWrapper.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,6 @@ class TerminalWidget;

namespace HackStudio {

class ProcessStateWidget;

class TerminalWrapper final : public GUI::Widget {
C_OBJECT(TerminalWrapper)
public:
Expand All @@ -50,7 +48,6 @@ class TerminalWrapper final : public GUI::Widget {
private:
explicit TerminalWrapper(bool user_spawned = true);

RefPtr<ProcessStateWidget> m_process_state_widget;
RefPtr<TerminalWidget> m_terminal_widget;
pid_t m_pid { -1 };
bool m_user_spawned { true };
Expand Down

0 comments on commit 4e0ab1d

Please sign in to comment.