On Friday, January 03, 2014 02:37:36 PM Rodney Rushing wrote:
> See if this one works better.
> 

Really neat :)

It's really useful to be able to select the pieces from a list. What I am 
wondering though is if it wouldn't be a more flexible and useful tool, if was 
less group-centered, that is a simple, sortable table (QTableView).

The columns would contain the
 * part description, possibly part number
 * part color
 * position and orientation 
 * building step number
 * group name

The use cases would be something like easily finding, selecting and bulk 
changing color/position/building step number/group name etc. of the relevant 
pieces. That would be easy if one could just sort based on each of the 
columns.

> I compile in VS2010 and whatever Ubuntu is giving me at the command line.  I 
notice that the error checking is not equivalent.  Even on windows I notice 
the warning levels are different for command line build and IDE build.  Some 
of the checks are mutually exclusive between Windows and Ubuntu so I can't get 
it to build warning free on both platforms.
> 

I would suggest fixing the following warnings nevertheless:


In file included from qt/lc_qgrouptree.cpp:2:0:
qt/lc_qgrouptree.h: In constructor 
‘lcQGroupTree::PieceIterator::PieceIterator(QList<QTreeWidgetItem*>&)’:
qt/lc_qgrouptree.h:94:27: warning: ‘lcQGroupTree::PieceIterator::m_list’ will 
be initialized after [-Wreorder]
   QList<QTreeWidgetItem*> m_list;
                           ^
qt/lc_qgrouptree.h:93:12: warning:   ‘int 
lcQGroupTree::PieceIterator::m_index’ [-Wreorder]
   int      m_index;
            ^
qt/lc_qgrouptree.cpp:86:1: warning:   when initialized here [-Wreorder]
 lcQGroupTree::PieceIterator::PieceIterator(QList<QTreeWidgetItem*>& list) :
 ^
In file included from qt/lc_qgrouptree.cpp:2:0:
qt/lc_qgrouptree.h: In constructor 
‘lcQGroupTree::GroupIterator::GroupIterator(QList<QTreeWidgetItem*>&)’:
qt/lc_qgrouptree.h:117:27: warning: ‘lcQGroupTree::GroupIterator::m_list’ will 
be initialized after [-Wreorder]
   QList<QTreeWidgetItem*> m_list;
                           ^
qt/lc_qgrouptree.h:116:12: warning:   ‘int 
lcQGroupTree::GroupIterator::m_index’ [-Wreorder]
   int      m_index;
            ^
qt/lc_qgrouptree.cpp:136:1: warning:   when initialized here [-Wreorder]
 lcQGroupTree::GroupIterator::GroupIterator(QList<QTreeWidgetItem*>& list) :
 ^

common/lc_model.cpp: In member function ‘void 
lcModel::UtilGetSelectedTopChildren(Group*&, Piece*&)’:
common/lc_model.cpp:430:35: warning: variable ‘pPrevPiece’ set but not used [-
Wunused-but-set-variable]
  for (Piece *pPiece = m_pPieces, *pPrevPiece = NULL, *pNextPiece = NULL; 
pPiece; pPiece = pNextPiece)


Cheers,


 Nicolas

-- 
Nicolas Guilbert
Maintainer of the Debian Leocad package

"Intelligence: property of a life form capable of outliving its planet of 
origin"

_______________________________________________
Leocad mailing list
[email protected]
https://list.gerf.org/listinfo/leocad

Reply via email to