Re: Review Request 107716: Fix generating kconfig skeletons with UrlList fields that have a default value
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107716/#review29012 --- Ship it! Ship It! - Albert Astals Cid On March 11, 2013, 3:16 a.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107716/ --- (Updated March 11, 2013, 3:16 a.m.) Review request for kdelibs. Description --- This patch changes the generated code so that if a property is UrlList in the kcfg file, the variable that will receive it when generating the skeleton is a KUrl::List instead a QStringList. This solves the following compiling error: aleix@kubuntu:~/projects/muon/build$ make [ 0%] Built target muonprivate_automoc [ 0%] Building CXX object libmuon/CMakeFiles/muonprivate.dir/MuonDataSources.o In file included from /usr/include/kcoreconfigskeleton.h:28:0, from /usr/include/kconfigskeleton.h:28, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.h:8, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:4: /usr/include/kurl.h: In constructor ‘MuonDataSources::MuonDataSources()’: /usr/include/kurl.h:1146:3: error: ‘KUrl::operator QString() const’ is private /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:38:105: error: within this context Diffs - kdecore/kconfig_compiler/kconfig_compiler.cpp 493c7d1 kdeui/tests/kconfig_compiler/CMakeLists.txt 28e916f kdeui/tests/kconfig_compiler/test12.cpp.ref PRE-CREATION kdeui/tests/kconfig_compiler/test12.h.ref PRE-CREATION kdeui/tests/kconfig_compiler/test12.kcfg PRE-CREATION kdeui/tests/kconfig_compiler/test12.kcfgc PRE-CREATION kdeui/tests/kconfig_compiler/test12main.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/107716/diff/ Testing --- Thanks, Aleix Pol Gonzalez
Re: Review Request 107716: Fix generating kconfig skeletons with UrlList fields that have a default value
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107716/#review29014 --- This review has been submitted with commit e02018941445df731873edb119b3290998375bdf by Aleix Pol to branch KDE/4.10. - Commit Hook On March 11, 2013, 3:16 a.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107716/ --- (Updated March 11, 2013, 3:16 a.m.) Review request for kdelibs. Description --- This patch changes the generated code so that if a property is UrlList in the kcfg file, the variable that will receive it when generating the skeleton is a KUrl::List instead a QStringList. This solves the following compiling error: aleix@kubuntu:~/projects/muon/build$ make [ 0%] Built target muonprivate_automoc [ 0%] Building CXX object libmuon/CMakeFiles/muonprivate.dir/MuonDataSources.o In file included from /usr/include/kcoreconfigskeleton.h:28:0, from /usr/include/kconfigskeleton.h:28, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.h:8, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:4: /usr/include/kurl.h: In constructor ‘MuonDataSources::MuonDataSources()’: /usr/include/kurl.h:1146:3: error: ‘KUrl::operator QString() const’ is private /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:38:105: error: within this context Diffs - kdecore/kconfig_compiler/kconfig_compiler.cpp 493c7d1 kdeui/tests/kconfig_compiler/CMakeLists.txt 28e916f kdeui/tests/kconfig_compiler/test12.cpp.ref PRE-CREATION kdeui/tests/kconfig_compiler/test12.h.ref PRE-CREATION kdeui/tests/kconfig_compiler/test12.kcfg PRE-CREATION kdeui/tests/kconfig_compiler/test12.kcfgc PRE-CREATION kdeui/tests/kconfig_compiler/test12main.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/107716/diff/ Testing --- Thanks, Aleix Pol Gonzalez
Re: Review Request 107716: Fix generating kconfig skeletons with UrlList fields that have a default value
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107716/ --- (Updated March 11, 2013, 6:42 p.m.) Status -- This change has been marked as submitted. Review request for kdelibs. Description --- This patch changes the generated code so that if a property is UrlList in the kcfg file, the variable that will receive it when generating the skeleton is a KUrl::List instead a QStringList. This solves the following compiling error: aleix@kubuntu:~/projects/muon/build$ make [ 0%] Built target muonprivate_automoc [ 0%] Building CXX object libmuon/CMakeFiles/muonprivate.dir/MuonDataSources.o In file included from /usr/include/kcoreconfigskeleton.h:28:0, from /usr/include/kconfigskeleton.h:28, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.h:8, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:4: /usr/include/kurl.h: In constructor ‘MuonDataSources::MuonDataSources()’: /usr/include/kurl.h:1146:3: error: ‘KUrl::operator QString() const’ is private /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:38:105: error: within this context Diffs - kdecore/kconfig_compiler/kconfig_compiler.cpp 493c7d1 kdeui/tests/kconfig_compiler/CMakeLists.txt 28e916f kdeui/tests/kconfig_compiler/test12.cpp.ref PRE-CREATION kdeui/tests/kconfig_compiler/test12.h.ref PRE-CREATION kdeui/tests/kconfig_compiler/test12.kcfg PRE-CREATION kdeui/tests/kconfig_compiler/test12.kcfgc PRE-CREATION kdeui/tests/kconfig_compiler/test12main.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/107716/diff/ Testing --- Thanks, Aleix Pol Gonzalez
Re: Review Request 107716: Fix generating kconfig skeletons with UrlList fields that have a default value
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107716/ --- (Updated March 11, 2013, 3:16 a.m.) Review request for kdelibs. Changes --- Included the unit test. Hope this can go in now. Description --- This patch changes the generated code so that if a property is UrlList in the kcfg file, the variable that will receive it when generating the skeleton is a KUrl::List instead a QStringList. This solves the following compiling error: aleix@kubuntu:~/projects/muon/build$ make [ 0%] Built target muonprivate_automoc [ 0%] Building CXX object libmuon/CMakeFiles/muonprivate.dir/MuonDataSources.o In file included from /usr/include/kcoreconfigskeleton.h:28:0, from /usr/include/kconfigskeleton.h:28, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.h:8, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:4: /usr/include/kurl.h: In constructor ‘MuonDataSources::MuonDataSources()’: /usr/include/kurl.h:1146:3: error: ‘KUrl::operator QString() const’ is private /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:38:105: error: within this context Diffs (updated) - kdecore/kconfig_compiler/kconfig_compiler.cpp 493c7d1 kdeui/tests/kconfig_compiler/CMakeLists.txt 28e916f kdeui/tests/kconfig_compiler/test12.cpp.ref PRE-CREATION kdeui/tests/kconfig_compiler/test12.h.ref PRE-CREATION kdeui/tests/kconfig_compiler/test12.kcfg PRE-CREATION kdeui/tests/kconfig_compiler/test12.kcfgc PRE-CREATION kdeui/tests/kconfig_compiler/test12main.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/107716/diff/ Testing --- Thanks, Aleix Pol Gonzalez
Re: Review Request 107716: Fix generating kconfig skeletons with UrlList fields that have a default value
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107716/#review28492 --- I'm getting the same error with an Url property. Can you please include the same fix for Url properties? - Chuso Perez On Dec. 14, 2012, 1:17 p.m., Aleix Pol Gonzalez wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107716/ --- (Updated Dec. 14, 2012, 1:17 p.m.) Review request for kdelibs. Description --- This patch changes the generated code so that if a property is UrlList in the kcfg file, the variable that will receive it when generating the skeleton is a KUrl::List instead a QStringList. This solves the following compiling error: aleix@kubuntu:~/projects/muon/build$ make [ 0%] Built target muonprivate_automoc [ 0%] Building CXX object libmuon/CMakeFiles/muonprivate.dir/MuonDataSources.o In file included from /usr/include/kcoreconfigskeleton.h:28:0, from /usr/include/kconfigskeleton.h:28, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.h:8, from /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:4: /usr/include/kurl.h: In constructor ‘MuonDataSources::MuonDataSources()’: /usr/include/kurl.h:1146:3: error: ‘KUrl::operator QString() const’ is private /home/aleix/projects/muon/build/libmuon/MuonDataSources.cpp:38:105: error: within this context Diffs - kdecore/kconfig_compiler/kconfig_compiler.cpp 4203d30 Diff: http://git.reviewboard.kde.org/r/107716/diff/ Testing --- Thanks, Aleix Pol Gonzalez