[QGIS-Developer] Segmentation fault with 'Create raster icons' on and loading a WMS... opinion to remove the preview feature?

[Luigi Pirelli]

FYI
hwo complementary PR to solve the issue:
https://github.com/qgis/QGIS/pull/5299 by Mathias
https://github.com/qgis/QGIS/pull/5302 by me
Luigi Pirelli

**************************************************************************************************
* Boundless QGIS Support/Development: lpirelli AT boundlessgeo DOT com
* LinkedIn: https://www.linkedin.com/in/luigipirelli
* Stackexchange: http://gis.stackexchange.com/users/19667/luigi-pirelli
* GitHub: https://github.com/luipir
* Mastering QGIS 2nd Edition:
* https://www.packtpub.com/big-data-and-business-intelligence/mastering-qgis-second-edition
**************************************************************************************************


On 4 October 2017 at 17:49, Matthias Kuhn <matthias at opengis.ch> wrote:
> Yes, kill it for WMS please
>
> On 10/04/2017 05:07 PM, Luigi Pirelli wrote:
>> or just skip preview only for WMS
>> Luigi Pirelli
>>
>> **************************************************************************************************
>> * Boundless QGIS Support/Development: lpirelli AT boundlessgeo DOT com
>> * LinkedIn: https://www.linkedin.com/in/luigipirelli
>> * Stackexchange: http://gis.stackexchange.com/users/19667/luigi-pirelli
>> * GitHub: https://github.com/luipir
>> * Mastering QGIS 2nd Edition:
>> * https://www.packtpub.com/big-data-and-business-intelligence/mastering-qgis-second-edition
>> **************************************************************************************************
>>
>>
>> On 4 October 2017 at 17:02, Luigi Pirelli <luipir at gmail.com> wrote:
>>> Hi
>>>
>>> related with issue: https://issues.qgis.org/issues/16803
>>> I strongly agree with Nyall, Pedro Venâncio and Giovanni Manghi to
>>> remove this feature.
>>> The rational is:
>>>
>>> 1) fix seems hard to find (QgsRasterLayer::previewAsImage is triggered
>>> 4/6 times and at aqvery refresh => not useful)... hard bu not
>>> impossible.
>>> 2) hard to find utility in a so little preview
>>> 3) who want to spend dev time on fixing it is welcome :)
>>> 4) Avioid crash is better than a having a not so important feature.
>>>
>>> I'll prepare a PR to remove only the option into "Settings ->
>>> Options... -> Canvas & Legend -> "Create raster icons (may be slow)"
>>> default is Off.
>>>
>>> any opinion?
>>>
>>> Luigi Pirelli
>>>
>>> **************************************************************************************************
>>> * Boundless QGIS Support/Development: lpirelli AT boundlessgeo DOT com
>>> * LinkedIn: https://www.linkedin.com/in/luigipirelli
>>> * Stackexchange: http://gis.stackexchange.com/users/19667/luigi-pirelli
>>> * GitHub: https://github.com/luipir
>>> * Mastering QGIS 2nd Edition:
>>> * https://www.packtpub.com/big-data-and-business-intelligence/mastering-qgis-second-edition
>>> **************************************************************************************************
>> _______________________________________________
>> QGIS-Developer mailing list
>> QGIS-Developer at lists.osgeo.org
>> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>>
> _______________________________________________
> QGIS-Developer mailing list
> QGIS-Developer at lists.osgeo.org
> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer