<table><tr><td style="">nicolasfella 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/D8966" rel="noreferrer">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/D8966#inline-40147" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">BackgroundService.java:294</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: #aa2211">.</span><span style="color: #354bb3">setSmallIcon</span><span style="color: #aa2211">(</span><span class="n">R</span><span style="color: #aa2211">.</span><span style="color: #354bb3">drawable</span><span style="color: #aa2211">.</span><span style="color: #354bb3">ic_notification</span><span style="color: #aa2211">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">                <span style="color: #aa2211">.</span><span style="color: #354bb3">setColor</span><span style="color: #aa2211">(</span><span class="n">getResources</span><span style="color: #aa2211">().</span><span style="color: #354bb3">getColor</span><span style="color: #aa2211">(</span><span class="n">R</span><span style="color: #aa2211">.</span><span style="color: #354bb3">color</span><span style="color: #aa2211">.</span><span style="color: #354bb3">primary</span><span style="color: #aa2211">))</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">                <span style="color: #aa2211">.</span><span style="color: #354bb3">setContentTitle</span><span style="color: #aa2211">(</span><span class="n">getString</span><span style="color: #aa2211">(</span><span class="n">R</span><span style="color: #aa2211">.</span><span style="color: #354bb3">string</span><span style="color: #aa2211">.</span><span style="color: #354bb3">kde_connect</span><span style="color: #aa2211">))</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Why set a explicit color? I would go with the default and remove the line</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/D8966#inline-40146" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">BackgroundService.java:314</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 class="n">startForeground</span><span style="color: #aa2211">(</span><span class="n">FOREGROUND_NOTIFICATION_ID</span><span style="color: #aa2211">,</span> <span class="n">notification</span><span style="color: #aa2211">.</span><span style="color: #354bb3">build</span><span style="color: #aa2211">());</span>
</div><div style="padding: 0 8px; margin: 0 4px; ">    <span style="color: #aa2211">}</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Also I expect a method called createForegroundNotification to do nothing but creating the notification. This has nothing to do with creating but actually using the the notification, thus is a side-effect which should be avoided.</p>

<p style="padding: 0; margin: 8px;">According to the docs this must be called after the service is started. Not sure if this is the case here (especially when createForegroundNotification is called from oncreate(). Maybe it should be moved to onStartCommand()</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8966" rel="noreferrer">https://phabricator.kde.org/D8966</a></div></div><br /><div><strong>To: </strong>mtijink, KDE Connect<br /><strong>Cc: </strong>nicolasfella<br /></div>