From 547fa73991ebf14d6a33bae3d955ebce6b8d74f2 Mon Sep 17 00:00:00 2001
From: Marcus Eggenberger <egs@quassel-irc.org>
Date: Sun, 10 Aug 2008 14:53:05 +0200
Subject: [PATCH] using a shared buffer for QTextBoundaryFinder

---
 src/qtui/chatlinemodelitem.cpp | 65 +++++++++++++++++++++++-----------
 src/qtui/chatlinemodelitem.h   | 29 ++++++++-------
 2 files changed, 59 insertions(+), 35 deletions(-)

diff --git a/src/qtui/chatlinemodelitem.cpp b/src/qtui/chatlinemodelitem.cpp
index 84832cdf..9b9e71c2 100644
--- a/src/qtui/chatlinemodelitem.cpp
+++ b/src/qtui/chatlinemodelitem.cpp
@@ -26,7 +26,23 @@
 #include "qtui.h"
 #include "uistyle.h"
 
-ChatLineModelItem::ChatLineModelItem(const Message &msg) : MessageModelItem(msg) {
+// This Struct is taken from Harfbuzz. We use it only to calc it's size.
+// we use a shared memory region so we do not have to malloc a buffer area for every line
+typedef struct {
+    /*HB_LineBreakType*/ unsigned lineBreakType  :2;
+    /*HB_Bool*/ unsigned whiteSpace              :1;     /* A unicode whitespace character, except NBSP, ZWNBSP */
+    /*HB_Bool*/ unsigned charStop                :1;     /* Valid cursor position (for left/right arrow) */
+    /*HB_Bool*/ unsigned wordBoundary            :1;
+    /*HB_Bool*/ unsigned sentenceBoundary        :1;
+    unsigned unused                  :2;
+} HB_CharAttributes_Dummy;
+
+unsigned char *ChatLineModelItem::TextBoundaryFinderBuffer = (unsigned char *)malloc(512 * sizeof(HB_CharAttributes_Dummy));
+int ChatLineModelItem::TextBoundaryFinderBufferSize = 512 * (sizeof(HB_CharAttributes_Dummy) / sizeof(unsigned char));
+
+ChatLineModelItem::ChatLineModelItem(const Message &msg)
+  : MessageModelItem(msg)
+{
   QtUiStyle::StyledMessage m = QtUi::style()->styleMessage(msg);
 
   _timestamp.plainText = m.timestamp.plainText;
@@ -42,40 +58,49 @@ ChatLineModelItem::ChatLineModelItem(const Message &msg) : MessageModelItem(msg)
 
 
 QVariant ChatLineModelItem::data(int column, int role) const {
-  const ChatLinePart *part;
+  const ChatLinePart *part = 0;
 
   switch(column) {
-    case ChatLineModel::TimestampColumn: part = &_timestamp; break;
-    case ChatLineModel::SenderColumn:    part = &_sender; break;
-    case ChatLineModel::ContentsColumn:      part = &_contents; break;
-    default: return MessageModelItem::data(column, role);
+  case ChatLineModel::TimestampColumn:
+    part = &_timestamp;
+    break;
+  case ChatLineModel::SenderColumn:
+    part = &_sender;
+    break;
+  case ChatLineModel::ContentsColumn:
+    part = &_contents;
+    break;
+  default:
+    return MessageModelItem::data(column, role);
   }
 
   switch(role) {
-    case ChatLineModel::DisplayRole:
-      return part->plainText;
-    case ChatLineModel::FormatRole:
-      return QVariant::fromValue<UiStyle::FormatList>(part->formatList);
-    case ChatLineModel::WrapListRole:
-      if(column != ChatLineModel::ContentsColumn) return QVariant();
-      return QVariant::fromValue<ChatLineModel::WrapList>(_wrapList);
+  case ChatLineModel::DisplayRole:
+    return part->plainText;
+  case ChatLineModel::FormatRole:
+    return QVariant::fromValue<UiStyle::FormatList>(part->formatList);
+  case ChatLineModel::WrapListRole:
+    if(column != ChatLineModel::ContentsColumn)
+      return QVariant();
+    return QVariant::fromValue<ChatLineModel::WrapList>(_wrapList);
   }
-
   return MessageModelItem::data(column, role);
 }
 
-bool ChatLineModelItem::setData(int column, const QVariant &value, int role) {
-  return false;
-}
-
 void ChatLineModelItem::computeWrapList() {
+  if(_contents.plainText.isEmpty())
+    return;
+
   enum Mode { SearchStart, SearchEnd };
 
   QList<ChatLineModel::Word> wplist;  // use a temp list which we'll later copy into a QVector for efficiency
-  QTextBoundaryFinder finder(QTextBoundaryFinder::Word, _contents.plainText);
+  // QTextBoundaryFinder finder(QTextBoundaryFinder::Word, _contents.plainText);
+  QTextBoundaryFinder finder(QTextBoundaryFinder::Word, _contents.plainText.unicode(), _contents.plainText.length(), TextBoundaryFinderBuffer, TextBoundaryFinderBufferSize);
+
   int idx;
   int oldidx = 0;
-  bool wordStart = false; bool wordEnd = false;
+  bool wordStart = false;
+  bool wordEnd = false;
   Mode mode = SearchEnd;
   ChatLineModel::Word word;
   word.start = 0;
diff --git a/src/qtui/chatlinemodelitem.h b/src/qtui/chatlinemodelitem.h
index a4df5203..a2ef9b9a 100644
--- a/src/qtui/chatlinemodelitem.h
+++ b/src/qtui/chatlinemodelitem.h
@@ -22,31 +22,30 @@
 #define CHATLINEMODELITEM_H_
 
 #include <QVector>
-#include <QPair>
 
 #include "chatlinemodel.h"
 #include "uistyle.h"
 
 class ChatLineModelItem : public MessageModelItem {
+public:
+  ChatLineModelItem(const Message &);
 
-  public:
+  virtual QVariant data(int column, int role) const;
+  virtual inline bool setData(int column, const QVariant &value, int role) { return false; }
 
-    ChatLineModelItem(const Message &);
-    //virtual ~ChatLineModelItem() {};
+private:
+  void computeWrapList();
 
-    virtual QVariant data(int column, int role) const;
-    virtual bool setData(int column, const QVariant &value, int role);
+  struct ChatLinePart {
+    QString plainText;
+    UiStyle::FormatList formatList;
+  };
+  ChatLinePart _timestamp, _sender, _contents;
 
-  private:
-    void computeWrapList();
+  ChatLineModel::WrapList _wrapList;
 
-    struct ChatLinePart {
-      QString plainText;
-      UiStyle::FormatList formatList;
-    };
-    ChatLinePart _timestamp, _sender, _contents;
-
-    ChatLineModel::WrapList _wrapList;
+  static unsigned char *TextBoundaryFinderBuffer;
+  static int TextBoundaryFinderBufferSize;
 };
 
 #endif
-- 
2.20.1