<font size="4"><font face="georgia,serif">I notice that if you run konsole from an xterm and Ctrl+C in xterm, the konsole history files are not removed.  I would suspect a computer crash or incorrectly shutdown would also leave the files.</font></font><div>

<font class="Apple-style-span" face="georgia, serif" size="4"><br></font><div><font size="4"><font face="georgia,serif">It would be nice to try to fix that and until then should we add "quitting Konsole cleanly." to the warning?</font></font></div>

<div><font size="4"><font face="georgia,serif"><br></font></font></div><div><font size="4"><font face="georgia,serif"> Kurt<br></font></font><br><div class="gmail_quote">On Mon, Apr 2, 2012 at 10:39 PM, Kurt Hindenburg <span dir="ltr"><<a href="mailto:kurt.hindenburg@gmail.com">kurt.hindenburg@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Git commit fe83cf054574ab5e849eb291ffe9b4ebd5c4b66a by Kurt Hindenburg.<br>
Committed on 03/04/2012 at 04:37.<br>
Pushed by hindenburg into branch 'master'.<br>
<br>
Add alert about unlimited scrollback saving to HD.<br>
<br>
Some people may not realize that when using 'unlimited scrollback' the<br>
scrollback is on the hard drive unencrypted.  The recent libvte<br>
'security' report brought this issue up.<br>
<br>
REVIEW: 104452<br>
<br>
M  +13   -1    src/HistorySizeWidget.cpp<br>
M  +27   -2    src/HistorySizeWidget.ui<br>
<br>
<a href="http://commits.kde.org/konsole/fe83cf054574ab5e849eb291ffe9b4ebd5c4b66a" target="_blank">http://commits.kde.org/konsole/fe83cf054574ab5e849eb291ffe9b4ebd5c4b66a</a><br>
<br>
diff --git a/src/HistorySizeWidget.cpp b/src/HistorySizeWidget.cpp<br>
index fcb131c..1e7670d 100644<br>
--- a/src/HistorySizeWidget.cpp<br>
+++ b/src/HistorySizeWidget.cpp<br>
@@ -36,6 +36,15 @@ HistorySizeWidget::HistorySizeWidget(QWidget* parent)<br>
     _ui = new Ui::HistorySizeWidget();<br>
     _ui->setupUi(this);<br>
<br>
+    _ui->unlimitedWarningWidget->setVisible(false);<br>
+    _ui->unlimitedWarningWidget->setWordWrap(true);<br>
+    _ui->unlimitedWarningWidget->setCloseButtonVisible(false);<br>
+    _ui->unlimitedWarningWidget->setMessageType(KMessageWidget::Warning);<br>
+    _ui->unlimitedWarningWidget->setText(i18nc("@info:status",<br>
+        "When using this option, the scrollback data will be written "<br>
+        "unencrypted to temporary files. Those temporary files will be "<br>
+        "deleted automatically when quitting Konsole."));<br>
+<br>
     // focus and select the spinner automatically when appropriate<br>
     _ui->fixedSizeHistoryButton->setFocusProxy(_ui->historyLineSpinner);<br>
     connect(_ui->fixedSizeHistoryButton , SIGNAL(clicked()) ,<br>
@@ -62,7 +71,9 @@ HistorySizeWidget::~HistorySizeWidget()<br>
<br>
 void HistorySizeWidget::buttonClicked(QAbstractButton*) const<br>
 {<br>
-    emit historyModeChanged(mode());<br>
+    Enum::HistoryModeEnum selectedMode = mode();<br>
+    _ui->unlimitedWarningWidget->setVisible(Enum::UnlimitedHistory == selectedMode);<br>
+    emit historyModeChanged(selectedMode);<br>
 }<br>
<br>
 void HistorySizeWidget::setMode(Enum::HistoryModeEnum aMode)<br>
@@ -74,6 +85,7 @@ void HistorySizeWidget::setMode(Enum::HistoryModeEnum aMode)<br>
     } else if (aMode == Enum::UnlimitedHistory) {<br>
         _ui->unlimitedHistoryButton->setChecked(true);<br>
     }<br>
+    _ui->unlimitedWarningWidget->setVisible(Enum::UnlimitedHistory == aMode);<br>
<br>
 }<br>
<br>
diff --git a/src/HistorySizeWidget.ui b/src/HistorySizeWidget.ui<br>
index abba938..076b575 100644<br>
--- a/src/HistorySizeWidget.ui<br>
+++ b/src/HistorySizeWidget.ui<br>
@@ -7,10 +7,16 @@<br>
     <x>0</x><br>
     <y>0</y><br>
     <width>400</width><br>
-    <height>86</height><br>
+    <height>143</height><br>
    </rect><br>
   </property><br>
-  <layout class="QVBoxLayout"><br>
+  <property name="sizePolicy"><br>
+   <sizepolicy hsizetype="Preferred" vsizetype="Fixed"><br>
+    <horstretch>0</horstretch><br>
+    <verstretch>0</verstretch><br>
+   </sizepolicy><br>
+  </property><br>
+  <layout class="QVBoxLayout" name="verticalLayout"><br>
    <item><br>
     <widget class="QRadioButton" name="noHistoryButton"><br>
      <property name="sizePolicy"><br>
@@ -29,6 +35,9 @@<br>
    </item><br>
    <item><br>
     <layout class="QHBoxLayout"><br>
+     <property name="sizeConstraint"><br>
+      <enum>QLayout::SetFixedSize</enum><br>
+     </property><br>
      <item><br>
       <widget class="QRadioButton" name="fixedSizeHistoryButton"><br>
        <property name="sizePolicy"><br>
@@ -89,6 +98,16 @@<br>
      </property><br>
     </widget><br>
    </item><br>
+   <item><br>
+    <widget class="KMessageWidget" name="unlimitedWarningWidget"><br>
+     <property name="sizePolicy"><br>
+      <sizepolicy hsizetype="Preferred" vsizetype="Fixed"><br>
+       <horstretch>0</horstretch><br>
+       <verstretch>0</verstretch><br>
+      </sizepolicy><br>
+     </property><br>
+    </widget><br>
+   </item><br>
   </layout><br>
  </widget><br>
  <customwidgets><br>
@@ -97,6 +116,12 @@<br>
    <extends>QSpinBox</extends><br>
    <header>knuminput.h</header><br>
   </customwidget><br>
+  <customwidget><br>
+   <class>KMessageWidget</class><br>
+   <extends>QFrame</extends><br>
+   <header>kmessagewidget.h</header><br>
+   <container>1</container><br>
+  </customwidget><br>
  </customwidgets><br>
  <resources/><br>
  <connections/><br>
</blockquote></div><br></div></div>