[vte/wip/egmont/bidi: 54/87] vla
- From: Egmont Koblinger <egmontkob src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [vte/wip/egmont/bidi: 54/87] vla
- Date: Tue, 9 Apr 2019 09:13:05 +0000 (UTC)
commit 02b4c91e6dd89a97ad0d7e839a52b071ad2bd995
Author: Egmont Koblinger <egmont gmail com>
Date: Wed Aug 29 22:54:38 2018 +0200
vla
src/bidi.cc | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
---
diff --git a/src/bidi.cc b/src/bidi.cc
index b31b0d2e..6a8dfb9d 100644
--- a/src/bidi.cc
+++ b/src/bidi.cc
@@ -291,7 +291,7 @@ vte::grid::row_t RingView::paragraph(vte::grid::row_t row)
unsigned int col;
/* The buffer size assumes that combining chars are omitted. It's an overkill, but convenient
solution. */
- // FIXME this is valid in C++, not just a gcc extension, correct? Or should we call g_newa()?
+ // FIXME VLA is a gcc extension, use g_newa() instead
FriBidiChar fribidi_chars[VTE_BIDI_PARAGRAPH_LENGTH_MAX * m_width];
/* Extract the paragraph's contents, omitting unused and fragment cells. */
@@ -332,7 +332,7 @@ vte::grid::row_t RingView::paragraph(vte::grid::row_t row)
}
/* Run the BiDi algorithm on the paragraph to get the embedding levels. */
- // FIXME this is valid in C++, not just a gcc extension, correct? Or should we call g_newa()?
+ // FIXME VLA is a gcc extension, use g_newa() instead
FriBidiCharType fribidi_chartypes[count];
FriBidiBracketType fribidi_brackettypes[count];
FriBidiLevel fribidi_levels[count];
@@ -384,7 +384,7 @@ vte::grid::row_t RingView::paragraph(vte::grid::row_t row)
break;
/* Map from FriBidi's to terminal's logical position, see the detailed explanation above. */
- // FIXME this is valid in C++, not just a gcc extension, correct? Or should we call g_newa()?
+ // FIXME VLA is a gcc extension, use g_newa() instead
FriBidiStrIndex fribidi_to_terminal[lines[line + 1] - lines[line]];
fl = 0;
for (tl = 0; tl < m_width && tl < row_data->len; tl++) {
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]