<table><tr><td style="">dfaure accepted this revision.<br />dfaure added a comment.<br />This revision is now accepted and ready to land.
</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/D5181" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I agree.<br />
The person who wrote this API doc was aiming for something slightly simpler (one less method to call), but my original design was for everyone to call setXMLFile, and it would make things a bit weird at the kxmlguiwindow level.</p>

<p>(I just don't agree with the #ifdef argument because one just call setXMLFile in all cases, the discussion is more about making things convenient for future new code)</p>

<p>Also note that lxr finds all calls to setXMLFile, including those in parts and plugins, while here we're only talking about kparts mainwindows. But yeah that was the initial design: calling setXMLFile everywhere, in mainwindows, parts and plugins. Then some people tried to make the API more compact in KXMLGuiWindow, but these KParts API remain basically unchanged.</p>

<p>Anyhow, feel free to ship it.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R306 KParts</div></div></div><br /><div><strong>BRANCH</strong><div><div>readdsetXMLFileNoteInDox</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5181" rel="noreferrer">https://phabricator.kde.org/D5181</a></div></div><br /><div><strong>To: </strong>kossebau, elvisangelaccio, Frameworks, dfaure<br /></div>