<table><tr><td style="">kossebau added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D21344">View Revision</a></tr></table><br /><div><div><p>The diff between the generated qmldir files and the manually written ones looks good. And also showed the one of plasmacore has some left-over.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21344#inline-119859">View Inline</a><span style="color: #4b4d51; font-weight: bold;">CMakeLists.txt:37</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    OBJECT_TYPES
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        BusyIndicator qml/BusyIndicator.qml
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        Button qml/Button.qml
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">All the repeated names make me wonder if the macro could be made even more nice by having it autoderive the type name from the file.  At least all the files in Plasma and the mojority in my own projects match this naming pattern.</p>

<p style="padding: 0; margin: 8px;">As that might blow up the macro code a bit, first waiting for principal feedback.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21344">https://phabricator.kde.org/D21344</a></div></div><br /><div><strong>To: </strong>kossebau, Plasma<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>