> On June 12, 2015, 2:14 a.m., Torsten Rahn wrote: > > src/lib/marble/MarbleLegendBrowser.cpp, line 411 > > <https://git.reviewboard.kde.org/r/124064/diff/1/?file=379492#file379492line411> > > > > Eeeeww - gutting the MarbleLegendBrowser and providing an alternative > > empty implementation this way looks quite scary to me. I wonder whether we > > should keep this at least in a different android specific cpp file? :-) > > Gábor Péterffy wrote: > Sure, I will update this patch after my exam on tuesday.
I once had a patch (and can still find it if needed) that cut off QWebView a bit "smaller". The idea was to get rid of it like this: class Foo: public #ifdef QT_NO_WEBKIT QWidget #else QWebView #endif { ... }; - Dennis ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124064/#review81401 ----------------------------------------------------------- On June 10, 2015, 7:35 p.m., Gábor Péterffy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124064/ > ----------------------------------------------------------- > > (Updated June 10, 2015, 7:35 p.m.) > > > Review request for Marble. > > > Repository: marble > > > Description > ------- > > It successfully compiles the source code for Android, but the final apk is > bad (fix will be in a following patch, also there can be some problems with > res/libs.xml and with the resources). > > So the only purpose of this patch to have a source code which can be compiled > for Android without compilation errors. > > Some instructions to compile it: > https://techbase.kde.org/Projects/Marble/AndroidCompiling > > > Diffs > ----- > > CMakeLists.txt de4a9dc39b948385bdde574dcdb673a9938a186f > data/lang/CMakeLists.txt e3c1fedb1d482a5573e3bb302fda6237b9893316 > src/apps/marble-mobile/CMakeLists.txt > 82031bda6df3402dc69a09a5847f0a3ae2530cb7 > src/apps/marble-mobile/MobileMainWindow.cpp > b2493f46135244daa131e197e3eb05bc7dc711b8 > src/lib/marble/MarbleLegendBrowser.h > 7e80c389f9e8f0fe3876dc4fe23d917e70a078d9 > src/lib/marble/MarbleLegendBrowser.cpp > f628b835534063aa459d67ddff10f33466ac59cd > src/lib/marble/MarbleWebView.h d564111586a25148a8391767c4f210e83f153951 > src/lib/marble/MarbleWebView.cpp b67a7fbc95642b93455c2f0717ac4b31d181e7af > src/lib/marble/PopupItem.cpp 8dd885c6efa4ed1c0981f835339b031720a24d01 > src/lib/marble/TinyWebBrowser.h 4e94eb5cf69710247d11358827ff7056f9253493 > src/lib/marble/TinyWebBrowser.cpp 52e88422c7bd3754a5235705b2de43ad77ed1437 > src/lib/marble/cloudsync/RouteItemDelegate.cpp > 9670f1e3ccb643bab5400e84e01129b9cc6826a4 > src/lib/marble/routing/RoutingInputWidget.cpp > 40c4aad9a93d3255a66fbfb98487401ee597321e > src/plugins/render/CMakeLists.txt bdfc085f1feca6f94011a3ae32b84a76bbc1602e > src/plugins/render/wikipedia/WikipediaItem.cpp > c708d443caf80ac2246f4c886197c109b8942bf1 > > Diff: https://git.reviewboard.kde.org/r/124064/diff/ > > > Testing > ------- > > Obvious crash without loading anything, but this is supposed to happen:) > > > Thanks, > > Gábor Péterffy > >
_______________________________________________ Marble-devel mailing list Marble-devel@kde.org https://mail.kde.org/mailman/listinfo/marble-devel