<table><tr><td style="">sredman requested changes to this revision.<br />sredman added inline comments.<br />This revision now requires changes to proceed.
</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/D17250">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/D17250#inline-94375">View Inline</a><span style="color: #4b4d51; font-weight: bold;">BluetoothPairingHandler.java:55</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; ">    <span style="color: #aa22ff">@Override</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);">    <span style="color: #aa4000">public</span> <span style="color: #aa4000">void</span> <span style="color: #004012">packageReceived</span><span style="color: #aa2211">(</span><span class="n">NetworkPacket</span> <span class="n">np</span><span style="color: #aa2211">)</span> <span class="bright"></span><span style="color: #aa4000"><span class="bright">throws</span></span><span class="bright"> </span><span class="n"><span class="bright">Exception</span></span><span class="bright"> </span><span style="color: #aa2211">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">public</span> <span style="color: #aa4000">void</span> <span style="color: #004012">packageReceived</span><span style="color: #aa2211">(</span><span class="n">NetworkPacket</span> <span class="n">np</span><span style="color: #aa2211">)</span> <span style="color: #aa2211">{</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Can you say why a lot of methods are no longer throwing? Was it unnecessary to begin with? In any case, it would be best if that change (exceptions) were separated from this change (header fixing)</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/D17250">https://phabricator.kde.org/D17250</a></div></div><br /><div><strong>To: </strong>brute4s99, KDE Connect, sredman<br /><strong>Cc: </strong>albertvaka, sredman, kdeconnect, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara, apol<br /></div>