Re: Review Request: Added include files for two widgets

2011-06-29 Thread Ingo Klöcker
On Monday 23 May 2011, Christoph Feck wrote:
  On May 23, 2011, 9:33 a.m., Christoph Feck wrote:
   It is not called RantingWidget, please correct it. Otherwise,
   good catch.
 
 Hm, why didn't I get John's mail ...

Because all Review Requests going to kde-core-devel (or any other KDE 
mailing list) have to be approved manually by the moderators and this 
might take a few hours...


Regards,
Ingo


signature.asc
Description: This is a digitally signed message part.


Re: Review Request: Added include files for two widgets

2011-05-23 Thread Commit Hook

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


This review has been submitted with commit 
5695a04308d33c505301517b5dae4c8c8e0f7168 by Jon Ander Peñalba.

- Commit


On May 22, 2011, 11:22 p.m., Jon Ander Peñalba wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/101421/
 ---
 
 (Updated May 22, 2011, 11:22 p.m.)
 
 
 Review request for kdelibs.
 
 
 Summary
 ---
 
 KRantingWidget and KShortcutWidget didn't have an include file with the same 
 name as the class, this patch simply adds this files.
 
 
 Diffs
 -
 
   includes/CMakeLists.txt 0a80219 
   includes/KRantingWidget PRE-CREATION 
   includes/KShortcutWidget PRE-CREATION 
 
 Diff: http://git.reviewboard.kde.org/r/101421/diff
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jon Ander
 




Re: Review Request: Added include files for two widgets

2011-05-23 Thread Christoph Feck

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


It is not called RantingWidget, please correct it. Otherwise, good catch.

- Christoph


On May 22, 2011, 11:22 p.m., Jon Ander Peñalba wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 http://git.reviewboard.kde.org/r/101421/
 ---
 
 (Updated May 22, 2011, 11:22 p.m.)
 
 
 Review request for kdelibs.
 
 
 Summary
 ---
 
 KRantingWidget and KShortcutWidget didn't have an include file with the same 
 name as the class, this patch simply adds this files.
 
 
 Diffs
 -
 
   includes/CMakeLists.txt 0a80219 
   includes/KRantingWidget PRE-CREATION 
   includes/KShortcutWidget PRE-CREATION 
 
 Diff: http://git.reviewboard.kde.org/r/101421/diff
 
 
 Testing
 ---
 
 
 Thanks,
 
 Jon Ander