<div dir="ltr"><div><div><div>Hi all, <br><br></div>Noel makes a good point (my initial was totally wrong - not sure why I wrote what I wrote - kids screaming in the background, sorry). <br><br></div>I'm a bit curious about the deprecation warnings: LocalOfficeConnection is being derived from a deprecated class, but is not itself deprecated. I'm a but confused how this makes any sense?<br>
<br></div>Does anyone know the reasoning here? I assumed that LocalOfficeConnection was deprecated also because of it, but for some reason wasn't being marked as such either :(<br><div><br>Chris<br></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Apr 16, 2013 at 8:14 PM, Noel Grandin (via Code Review) <span dir="ltr"><<a href="mailto:gerrit@gerrit.libreoffice.org" target="_blank">gerrit@gerrit.libreoffice.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Noel Grandin has posted comments on this change.<br>
<br>
Change subject: Deprecated warnings disabled for Office Beans.<br>
......................................................................<br>
<br>
<br>
Patch Set 1: This need some tweaks before it is merged<br>
<br>
Your commit message doesn't make sense - the classes using the deprecated classes are not themselves marked as deprecated.<br>
<br>
If they were, the warnings would not be there.<br>
<br>
For example, if  LocalOfficeConnection was marked @deprecated, then you would not need the @SuppressWarnings.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
To view, visit <a href="https://gerrit.libreoffice.org/3407" target="_blank">https://gerrit.libreoffice.org/3407</a><br>
To unsubscribe, visit <a href="https://gerrit.libreoffice.org/settings" target="_blank">https://gerrit.libreoffice.org/settings</a><br>
<br>
Gerrit-MessageType: comment<br>
Gerrit-Change-Id: Ic49101f362dbff33afba63fb96b36ae88d3633f5<br>
Gerrit-PatchSet: 1<br>
Gerrit-Project: core<br>
Gerrit-Branch: master<br>
Gerrit-Owner: Chris Sherlock <<a href="mailto:chris.sherlock79@gmail.com">chris.sherlock79@gmail.com</a>><br>
Gerrit-Reviewer: Noel Grandin <<a href="mailto:noelgrandin@gmail.com">noelgrandin@gmail.com</a>><br>
</font></span></blockquote></div><br></div>