Review Request 117952: Fix KIO::suggestName() for the case of leading dots, and no other dot.

2014-05-02 Thread David Faure

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

Review request for KDE Frameworks and Kevin Funk.


Repository: kio


Description
---

Fix KIO::suggestName() for the case of leading dots, and no other dot.

We don't want to change the extension in that case.

With a unittest for that method, finally.


Diffs
-

  autotests/globaltest.h b955329cf7d5841d9f9c1230e41d1261a268420e 
  autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 
  src/core/global.cpp b6f523d66ef5b9308f615c485d6dcf3069b26ace 

Diff: https://git.reviewboard.kde.org/r/117952/diff/


Testing
---


Thanks,

David Faure

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 117952: Fix KIO::suggestName() for the case of leading dots, and no other dot.

2014-05-02 Thread Kevin Funk

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117952/#review57150
---

Ship it!


LGTM

- Kevin Funk


On May 2, 2014, 4:12 p.m., David Faure wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://git.reviewboard.kde.org/r/117952/
 ---
 
 (Updated May 2, 2014, 4:12 p.m.)
 
 
 Review request for KDE Frameworks and Kevin Funk.
 
 
 Repository: kio
 
 
 Description
 ---
 
 Fix KIO::suggestName() for the case of leading dots, and no other dot.
 
 We don't want to change the extension in that case.
 
 With a unittest for that method, finally.
 
 
 Diffs
 -
 
   autotests/globaltest.h b955329cf7d5841d9f9c1230e41d1261a268420e 
   autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 
   src/core/global.cpp b6f523d66ef5b9308f615c485d6dcf3069b26ace 
 
 Diff: https://git.reviewboard.kde.org/r/117952/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 David Faure
 


___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 117952: Fix KIO::suggestName() for the case of leading dots, and no other dot.

2014-05-02 Thread David Faure

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

(Updated May 2, 2014, 4:30 p.m.)


Status
--

This change has been marked as submitted.


Review request for KDE Frameworks and Kevin Funk.


Repository: kio


Description
---

Fix KIO::suggestName() for the case of leading dots, and no other dot.

We don't want to change the extension in that case.

With a unittest for that method, finally.


Diffs
-

  autotests/globaltest.h b955329cf7d5841d9f9c1230e41d1261a268420e 
  autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 
  src/core/global.cpp b6f523d66ef5b9308f615c485d6dcf3069b26ace 

Diff: https://git.reviewboard.kde.org/r/117952/diff/


Testing
---


Thanks,

David Faure

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel