Details
-
Task
-
Resolution: Done
-
P3: Somewhat important
-
None
-
None
-
89
-
Da Vinci sprint 10
Description
The implementation of Node and its heirs is monolithic. This makes it hard to get an overview, introduces needless cross-dependencies (if you need one type node somewhere, you get them all) that make unit testing unnecessarily complicated, and slows down static code analysis to the point where it sometimes halts. Chop the implementation into chunks of a reasonable size.
Attachments
For Gerrit Dashboard: QTBUG-84578 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
302428,5 | QDoc: Alphabetize forward declarations in Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302429,4 | QDoc: Alphabetize project file | dev | qt/qttools | Status: MERGED | +2 | 0 |
302430,7 | QDoc: Extract EnumItem from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302431,8 | QDoc: Extract VariableNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302432,4 | QDoc: Minor cleanup in docbookgenerator | dev | qt/qttools | Status: MERGED | +2 | 0 |
302434,5 | QDoc: Extract PropertyNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302435,6 | QDoc: Extract FunctionNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302437,5 | QDoc: Extract SharedCommentNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302438,7 | QDoc: Extract TypedefNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302440,7 | QDoc: Extract EnumNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302441,10 | QDoc: Extract QmlPropertyNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302450,11 | QDoc: Extract enum Access from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302451,12 | QDoc: Extract RelatedClass from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302454,5 | QDoc: Move method definition | dev | qt/qttools | Status: MERGED | +2 | 0 |
302455,7 | QDoc: Extract UsingClause from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302456,1 | WIP: QDoc: Extract aggregates from Node | dev | qt/qttools | Status: ABANDONED | -2 | 0 |
302458,6 | QDoc: Extract ImportRec from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302463,6 | QDoc: Extract QmlTypeNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302464,6 | QDoc: Extract HeaderNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302466,5 | QDoc: Extract ClassNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302467,5 | QDoc: Extract NamespaceNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302468,5 | QDoc: Extract ProxyNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302469,6 | QDoc: Extract CollectionNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302470,3 | QDoc: Extract ExampleNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302471,3 | QDoc: Extract ExternalPageNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302480,4 | QDoc: Extract Aggregate from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302482,4 | QDoc: Extract PageNode from Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302484,4 | QDoc: Clean up Node's ctor and dtor | dev | qt/qttools | Status: MERGED | +2 | 0 |
302487,7 | QDoc: Minor cleanup in Node | dev | qt/qttools | Status: MERGED | +2 | 0 |
302488,4 | QDoc: Clean up members in Node | dev | qt/qttools | Status: MERGED | +2 | 0 |