Opened 16 years ago

Closed 16 years ago

#1234 closed bug (fixed)

Suspicious retrival of active layer in dxf2shp converter

Reported by: mloskot Owned by: nobody
Priority: minor: annoyance Milestone:
Component: C++ Plugins Version: Trunk
Keywords: dxf2shp Cc:
Must Fix for Release: No Platform: OS X
Platform Version: 10.5 Awaiting user input: no

Description

In file dxf2shpconverter.cpp, lines 119-121, there are some actions implemented that seem to be garbage and never used:

QgsMapLayer *layer = mQGisIface->activeLayer();
if (layer)
    QgsVectorLayer *lyr = (QgsVectorLayer*)layer;

Or, it's a kind of logical error (cast and assignment to lyr object in local scope of if-condition) and the idea was to use the retrieved active layer but bits of usage are missing.

Change History (2)

comment:1 by mloskot, 16 years ago

Sorry, the link to file is broken, here is corrected version: dxf2shpconverter.cpp

comment:2 by jef, 16 years ago

Resolution: fixed
Status: newclosed

thanks. fixed in r9114

Note: See TracTickets for help on using tickets.