Skip to content

Commit 0cd4550

Browse files
committed
Remove Unnecessary Delegate Method
1 parent f245eaa commit 0cd4550

4 files changed

Lines changed: 11 additions & 12 deletions

File tree

CodeEdit/Features/TerminalEmulator/Views/CEActiveTaskTerminalView.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ class CEActiveTaskTerminalView: CELocalShellTerminalView {
5454
self.enableOutput = false
5555
let chunks = string.split(separator: ";")
5656
guard chunks.count == 2, let code = Int32(chunks[1]) else { return }
57-
self.processDelegate?.userProcessTerminated(exitCode: code)
57+
5858
if self.activeTask.status == .running {
5959
self.activeTask.handleProcessFinished(terminationStatus: code)
6060
}

CodeEdit/Features/TerminalEmulator/Views/CELocalShellTerminalView.swift

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,10 +42,6 @@ protocol CELocalShellTerminalViewDelegate: AnyObject {
4242
/// - Parameter exitCode: the exit code returned by the process, or nil if this was an error caused during
4343
/// the IO reading/writing
4444
func processTerminated(source: TerminalView, exitCode: Int32?)
45-
46-
/// Invoked when the shell integration notifies the terminal that a user-executed command has finished with an exit
47-
/// code.
48-
func userProcessTerminated(exitCode: Int32)
4945
}
5046

5147
// MARK: - CELocalShellTerminalView

CodeEdit/Features/TerminalEmulator/Views/TerminalEmulatorView+Coordinator.swift

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,12 @@ extension TerminalEmulatorView {
1313
private let terminalID: UUID
1414
public var onTitleChange: (_ title: String) -> Void
1515

16-
init(terminalID: UUID, onTitleChange: @escaping (_ title: String) -> Void) {
16+
var mode: TerminalMode
17+
18+
init(terminalID: UUID, mode: TerminalMode, onTitleChange: @escaping (_ title: String) -> Void) {
1719
self.terminalID = terminalID
1820
self.onTitleChange = onTitleChange
21+
self.mode = mode
1922
super.init()
2023
}
2124

@@ -31,11 +34,11 @@ extension TerminalEmulatorView {
3134
guard let exitCode else {
3235
return
3336
}
34-
source.feed(text: "Exit code: \(exitCode)\n\r\n")
35-
source.feed(text: "To open a new session, create a new terminal tab.")
36-
TerminalCache.shared.removeCachedView(terminalID)
37+
if case .shell = mode {
38+
source.feed(text: "Exit code: \(exitCode)\n\r\n")
39+
source.feed(text: "To open a new session, create a new terminal tab.")
40+
TerminalCache.shared.removeCachedView(terminalID)
41+
}
3742
}
38-
39-
func userProcessTerminated(exitCode: Int32) { }
4043
}
4144
}

CodeEdit/Features/TerminalEmulator/Views/TerminalEmulatorView.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,6 +218,6 @@ struct TerminalEmulatorView: NSViewRepresentable {
218218
}
219219

220220
func makeCoordinator() -> Coordinator {
221-
Coordinator(terminalID: terminalID, onTitleChange: onTitleChange)
221+
Coordinator(terminalID: terminalID, mode: mode, onTitleChange: onTitleChange)
222222
}
223223
}

0 commit comments

Comments
 (0)