<div dir="ltr">Hi!<div><br></div><div style>I am not sure but I think what Pirmin meant here was that there was no warning _message_.</div><div style>We all know that a lot of functions have been declared deprecated since a very long time. But without knowing or getting a warning message that THIS function is deprecated, I agree with Pirmin that noone can see the deprecation until reading the source or .. until it is cleaned out.</div>
<div style>Nevertheless it is good to clean the code (otherwise we would collect deprecated functions forever).. </div><div style>So just as a reminder here to spit out warning messages for deprecated functions next time .. </div>
<div style>Whenever this will be .. I hope not too soon ;)</div><div style><br></div><div style>kind regards</div><div style>Werner</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 24, 2013 at 7:40 AM, Alexander Bruy <span dir="ltr"><<a href="mailto:alexander.bruy@gmail.com" target="_blank">alexander.bruy@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
please note that most of this methods were marked as deprecated for<br>
a long time. Some of them are deprecated since QGIS 1.6 or even<br>
earlier.<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, 24 Jan 2013 00:57:26 +0100<br>
Pirmin Kalberer <<a href="mailto:pi_ml@sourcepole.com">pi_ml@sourcepole.com</a>> wrote:<br>
<br>
> Hi all,<br>
><br>
> I found the time for an OpenLayers plugin release with merged pull requests<br>
> for Stamen map support and fixes for Python API breaks in master branch (See<br>
> <a href="https://twitter.com/PirminKalberer/status/294226472707715072" target="_blank">https://twitter.com/PirminKalberer/status/294226472707715072</a> for credits).<br>
> The second point was quite annoying for many users and myself. It is a really<br>
> bad practice to break the API without deprecation messages when calling these<br>
> methods. There should be enough time (at least one minor version) for<br>
> developers for updating their plugins. In this case it would have been easy to<br>
> keep the old API and add a deprecation message similar to<br>
> <a href="http://code.activestate.com/recipes/391367-deprecated/" target="_blank">http://code.activestate.com/recipes/391367-deprecated/</a><br>
><br>
> Regards<br>
> Pirmin<br>
><br>
> --<br>
> Pirmin Kalberer<br>
> Sourcepole  -  Linux & Open Source Solutions<br>
> <a href="http://www.sourcepole.com" target="_blank">http://www.sourcepole.com</a><br>
><br>
> _______________________________________________<br>
> Qgis-developer mailing list<br>
> <a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Alexander Bruy<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</div></div></blockquote></div><br></div>