<table><tr><td style="">abalaji added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D16425">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D16425#inline-121266">View Inline</a><span style="color: #4b4d51; font-weight: bold;">suspendsessionconfig.cpp:122</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">int</span> <span class="n">comboBoxMaxWidth</span> <span style="color: #aa2211">=</span> <span style="color: #601200">300</span><span class="p">;</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">if</span> <span class="p">(</span><span class="n">m_suspendThenHibernateEnabled</span><span class="p">)</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">comboBoxMaxWidth</span> <span style="color: #aa2211">=</span> <span class="n">qMax</span><span class="p">(</span><span class="n">comboBoxMaxWidth</span><span class="p">,</span> <span class="n">m_suspendThenHibernateEnabled</span><span style="color: #aa2211">-></span><span class="n">sizeHint</span><span class="p">().</span><span class="n">width</span><span class="p">());</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">After doing the above, this if block can be merged with the above if block, if you know what I mean.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D16425#inline-120916">View Inline</a><span style="color: #4b4d51; font-weight: bold;">avaldes</span> wrote in <span style="color: #4b4d51; font-weight: bold;">suspendsessionconfig.cpp:106</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Done on purpose to change the UI:</p>

<p style="padding: 0; margin: 8px;"><a href="https://phabricator.kde.org/F6822188" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6822188: suspendThenHibernate.png</a></p>

<p style="padding: 0; margin: 8px;">Automatically, m_comboBox, m_idleTime</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Oh i see, that actually makes sense</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D16425#inline-120918">View Inline</a><span style="color: #4b4d51; font-weight: bold;">avaldes</span> wrote in <span style="color: #4b4d51; font-weight: bold;">suspendsessionconfig.cpp:121</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">yeah it's necessary, without it powerdevil crashes at launch</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Yeah it crashes because you're probably not doing a null check somewhere. I would suggest initializing <tt style="background: #ebebeb; font-size: 13px;">m_suspendThenHibernateEnabled </tt> to nullptr in the ctor, moving line 79 into this if block right above, and getting rid of this else block altogether, that way you don't wastefully allocate the object. After this, check for nullptr in every place you use <tt style="background: #ebebeb; font-size: 13px;">m_suspendThenHibernateEnabled</tt>.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R122 Powerdevil</div></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D16425_1</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16425">https://phabricator.kde.org/D16425</a></div></div><br /><div><strong>To: </strong>avaldes, broulik, ngraham<br /><strong>Cc: </strong>jobauer, reverendhomer, meven, soriano, abalaji, graesslin, ngraham, plasma-devel, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>