[PATCH] Add signals support to kconfig_compiler

Michael Brade brade at kde.org
Thu Feb 2 13:49:07 GMT 2006


On Tuesday 31 January 2006 16:03, Michaël Larouche wrote:
> Patch included also fix to reference files(to make tests pass), the new
> test files, modifications to build system (automake and CMake), addition to
> the Doxygen files.
I noticed that you touched some files (test?.h.ref mainly) only to change the 
coding style by adding { and } to one-line ifs and an additional newline at 
the end of the file. But you are adding new code like 

+void TestSignal::usrWriteConfig()
+{
+  if ( mSettingsChanged & signalEmoticonSettingsChanged ) 
+    emit emoticonSettingsChanged();
+
+  if ( mSettingsChanged & signalStyleChanged ) 
+    emit styleChanged(mStylePath, mStyleCSSVariant);

so I would suggest to leave the old files as they are. Other than that it 
looks great to me!

-- 
Michael Brade;                 KDE Developer, Student of Computer Science
  |-mail: echo brade !#|tr -d "c oh"|s\e\d 's/e/\@/2;s/$/.org/;s/bra/k/2'
  °--web: http://www.kde.org/people/michaelb.html

KDE 4: Beyond Your Expectations
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20060202/3e193c97/attachment.sig>


More information about the kde-core-devel mailing list