summaryrefslogtreecommitdiff
path: root/Libraries
diff options
context:
space:
mode:
Diffstat (limited to 'Libraries')
-rw-r--r--Libraries/LibLine/Editor.cpp27
-rw-r--r--Libraries/LibLine/Editor.h17
-rw-r--r--Libraries/LibLine/StringMetrics.h49
-rw-r--r--Libraries/LibLine/SuggestionDisplay.h6
-rw-r--r--Libraries/LibLine/XtermSuggestionDisplay.cpp14
5 files changed, 86 insertions, 27 deletions
diff --git a/Libraries/LibLine/Editor.cpp b/Libraries/LibLine/Editor.cpp
index 892d9a6880..48226094af 100644
--- a/Libraries/LibLine/Editor.cpp
+++ b/Libraries/LibLine/Editor.cpp
@@ -56,7 +56,7 @@ Editor::Editor(Configuration configuration)
m_num_columns = ws.ws_col;
m_num_lines = ws.ws_row;
}
- m_suggestion_display = make<XtermSuggestionDisplay>(m_num_lines, m_num_columns);
+ m_suggestion_display = make<XtermSuggestionDisplay>(m_num_lines, m_num_columns, m_cached_prompt_metrics);
}
Editor::~Editor()
@@ -304,6 +304,7 @@ auto Editor::get_line(const String& prompt) -> Result<String, Editor::Error>
if (m_finish) {
m_finish = false;
+ reposition_cursor(true);
printf("\n");
fflush(stdout);
auto string = line();
@@ -950,6 +951,12 @@ void Editor::recalculate_origin()
}
void Editor::cleanup()
{
+ auto current_buffer_metrics = actual_rendered_string_metrics(buffer_view());
+ auto new_lines = current_prompt_metrics().lines_with_addition(current_buffer_metrics, m_num_columns);
+ auto shown_lines = num_lines();
+ if (new_lines < shown_lines)
+ m_extra_forward_lines = max(shown_lines - new_lines, m_extra_forward_lines);
+
VT::move_relative(-m_extra_forward_lines, m_pending_chars.size() - m_chars_inserted_in_the_middle);
auto current_line = cursor_line();
@@ -1105,14 +1112,22 @@ void Editor::strip_styles(bool strip_anchored)
m_refresh_needed = true;
}
-void Editor::reposition_cursor()
+void Editor::reposition_cursor(bool to_end)
{
- m_drawn_cursor = m_cursor;
+ auto cursor = m_cursor;
+ auto saved_cursor = m_cursor;
+ if (to_end)
+ cursor = m_buffer.size();
+
+ m_cursor = cursor;
+ m_drawn_cursor = cursor;
auto line = cursor_line() - 1;
auto column = offset_in_line();
VT::move_absolute(line + m_origin_row, column + m_origin_column);
+
+ m_cursor = saved_cursor;
}
void VT::move_absolute(u32 row, u32 col)
@@ -1307,7 +1322,7 @@ void VT::clear_to_end_of_line()
fflush(stdout);
}
-Editor::StringMetrics Editor::actual_rendered_string_metrics(const StringView& string) const
+StringMetrics Editor::actual_rendered_string_metrics(const StringView& string) const
{
size_t length { 0 };
StringMetrics metrics;
@@ -1331,7 +1346,7 @@ Editor::StringMetrics Editor::actual_rendered_string_metrics(const StringView& s
return metrics;
}
-Editor::StringMetrics Editor::actual_rendered_string_metrics(const Utf32View& view) const
+StringMetrics Editor::actual_rendered_string_metrics(const Utf32View& view) const
{
size_t length { 0 };
StringMetrics metrics;
@@ -1557,7 +1572,7 @@ void Editor::readjust_anchored_styles(size_t hint_index, ModificationKind modifi
}
}
-size_t Editor::StringMetrics::lines_with_addition(const StringMetrics& offset, size_t column_width) const
+size_t StringMetrics::lines_with_addition(const StringMetrics& offset, size_t column_width) const
{
size_t lines = 0;
diff --git a/Libraries/LibLine/Editor.h b/Libraries/LibLine/Editor.h
index 50c73e1482..2f9fb4693e 100644
--- a/Libraries/LibLine/Editor.h
+++ b/Libraries/LibLine/Editor.h
@@ -41,6 +41,7 @@
#include <LibCore/Notifier.h>
#include <LibCore/Object.h>
#include <LibLine/Span.h>
+#include <LibLine/StringMetrics.h>
#include <LibLine/Style.h>
#include <LibLine/SuggestionDisplay.h>
#include <LibLine/SuggestionManager.h>
@@ -104,20 +105,6 @@ public:
const Vector<String>& history() const { return m_history; }
void register_character_input_callback(char ch, Function<bool(Editor&)> callback);
- struct StringMetrics {
- Vector<size_t> line_lengths;
- size_t total_length { 0 };
- size_t max_line_length { 0 };
-
- size_t lines_with_addition(const StringMetrics& offset, size_t column_width) const;
- void reset()
- {
- line_lengths.clear();
- total_length = 0;
- max_line_length = 0;
- line_lengths.append(0);
- }
- };
StringMetrics actual_rendered_string_metrics(const StringView&) const;
StringMetrics actual_rendered_string_metrics(const Utf32View&) const;
@@ -313,7 +300,7 @@ private:
bool should_break_token(Vector<u32, 1024>& buffer, size_t index);
void recalculate_origin();
- void reposition_cursor();
+ void reposition_cursor(bool to_end = false);
struct CodepointRange {
size_t start { 0 };
diff --git a/Libraries/LibLine/StringMetrics.h b/Libraries/LibLine/StringMetrics.h
new file mode 100644
index 0000000000..9fdd8ee719
--- /dev/null
+++ b/Libraries/LibLine/StringMetrics.h
@@ -0,0 +1,49 @@
+/*
+ * Copyright (c) 2020, The SerenityOS developers.
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright notice, this
+ * list of conditions and the following disclaimer.
+ *
+ * 2. Redistributions in binary form must reproduce the above copyright notice,
+ * this list of conditions and the following disclaimer in the documentation
+ * and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#pragma once
+
+#include <AK/Types.h>
+#include <AK/Vector.h>
+
+namespace Line {
+
+struct StringMetrics {
+ Vector<size_t> line_lengths;
+ size_t total_length { 0 };
+ size_t max_line_length { 0 };
+
+ size_t lines_with_addition(const StringMetrics& offset, size_t column_width) const;
+ void reset()
+ {
+ line_lengths.clear();
+ total_length = 0;
+ max_line_length = 0;
+ line_lengths.append(0);
+ }
+};
+
+}
diff --git a/Libraries/LibLine/SuggestionDisplay.h b/Libraries/LibLine/SuggestionDisplay.h
index 346ee4bc59..80606bc5fc 100644
--- a/Libraries/LibLine/SuggestionDisplay.h
+++ b/Libraries/LibLine/SuggestionDisplay.h
@@ -28,6 +28,7 @@
#include <AK/Forward.h>
#include <AK/String.h>
+#include <LibLine/StringMetrics.h>
#include <LibLine/SuggestionManager.h>
#include <stdlib.h>
@@ -61,9 +62,10 @@ protected:
class XtermSuggestionDisplay : public SuggestionDisplay {
public:
- XtermSuggestionDisplay(size_t lines, size_t columns)
+ XtermSuggestionDisplay(size_t lines, size_t columns, const StringMetrics& prompt_metrics)
: m_num_lines(lines)
, m_num_columns(columns)
+ , m_prompt_metrics(prompt_metrics)
{
}
virtual ~XtermSuggestionDisplay() override { }
@@ -92,7 +94,7 @@ private:
size_t m_num_lines { 0 };
size_t m_num_columns { 0 };
size_t m_prompt_lines_at_suggestion_initiation { 0 };
- size_t m_prompt_length { 0 };
+ const StringMetrics& m_prompt_metrics;
struct PageRange {
size_t start;
diff --git a/Libraries/LibLine/XtermSuggestionDisplay.cpp b/Libraries/LibLine/XtermSuggestionDisplay.cpp
index 7cfff9999b..b908056b13 100644
--- a/Libraries/LibLine/XtermSuggestionDisplay.cpp
+++ b/Libraries/LibLine/XtermSuggestionDisplay.cpp
@@ -52,7 +52,11 @@ void XtermSuggestionDisplay::display(const SuggestionManager& manager)
VT::restore_cursor();
auto spans_entire_line { false };
- auto max_line_count = (m_prompt_length + longest_suggestion_length + m_num_columns - 1) / m_num_columns;
+ Vector<size_t> lines;
+ for (size_t i = 0; i < m_prompt_lines_at_suggestion_initiation - 1; ++i)
+ lines.append(0);
+ lines.append(longest_suggestion_length);
+ auto max_line_count = StringMetrics { move(lines) }.lines_with_addition({ { 0 } }, m_num_columns);
if (longest_suggestion_length >= m_num_columns - 2) {
spans_entire_line = true;
// We should make enough space for the biggest entry in
@@ -126,9 +130,9 @@ void XtermSuggestionDisplay::display(const SuggestionManager& manager)
if (spans_entire_line) {
num_printed += m_num_columns;
- fprintf(stderr, "%s", suggestion.text_string.characters());
+ fprintf(stdout, "%s", suggestion.text_string.characters());
} else {
- fprintf(stderr, "%-*s", static_cast<int>(longest_suggestion_byte_length) + 2, suggestion.text_string.characters());
+ fprintf(stdout, "%-*s", static_cast<int>(longest_suggestion_byte_length) + 2, suggestion.text_string.characters());
num_printed += longest_suggestion_length + 2;
}
@@ -153,6 +157,7 @@ void XtermSuggestionDisplay::display(const SuggestionManager& manager)
if (string.length() > m_num_columns - 1) {
// This would overflow into the next line, so just don't print an indicator.
+ fflush(stdout);
return;
}
@@ -160,8 +165,9 @@ void XtermSuggestionDisplay::display(const SuggestionManager& manager)
VT::apply_style({ Style::Background(Style::XtermColor::Green) });
fputs(string.characters(), stdout);
VT::apply_style(Style::reset_style());
- fflush(stdout);
}
+
+ fflush(stdout);
}
bool XtermSuggestionDisplay::cleanup()