Review Request: KMainWindow: fix position support in --geometry option

[Dominik Haumann]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107023/
-----------------------------------------------------------

(Updated Oct. 24, 2012, 9:27 p.m.)


Review request for kdelibs.


Changes
-------

Adapt description before committing the patch. Thanks for the review.


Description (updated)
-------

KDE applications support the option: --geometry <width>x<height>{+-}<xpos>{+-}<ypos>
Example: kwrite --geometry 800x600+100+100

While the width and height arguments work, the x/y positions only work if they are negative. The code was added 11 years ago with commit d3188ee411df716b20e10fcdaa9dbc506cc31759 and probably never worked. This patch tries to fix it.

In KDE 5, support for --geometry was removed with commit fbc835090758581ba57ffbd210c00e60b1e1742f in the frameworks branch. Still having a working version for the KDE 4 lifetime would be nice imo :-)

Related bug reports:
- https://bugs.kde.org/show_bug.cgi?id=147094
- https://bugs.kde.org/show_bug.cgi?id=165355


This addresses bug 304279.
    http://bugs.kde.org/show_bug.cgi?id=304279


Diffs
-----

  kdeui/widgets/kmainwindow.cpp 86ccab8 

Diff: http://git.reviewboard.kde.org/r/107023/diff/


Testing
-------

Not tested, since I have no KDE4 kdelibs build. It's very much appreciated, if someone else can test this.


Thanks,

Dominik Haumann

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20121024/c115313e/attachment.htm>