apps/tuxbox/neutrino/src/neutrino.cpp missing include

[idl0r]

build log:

neutrino.cpp: In member function `int CNeutrinoApp::loadSetup()':
neutrino.cpp:765: error: `FILEBROWSER_NUMBER_OF_SORT_VARIANTS' was not declared in this scope
neutrino.cpp:765: warning: unused variable 'FILEBROWSER_NUMBER_OF_SORT_VARIANTS'
neutrino.cpp:2915:2: warning: #warning fixme sectionsd should deliver data in UTF-8 format
neutrino.cpp:3103:2: warning: #warning TODO: it might make some sense to have some log-file (but where do we store this information? nfs/flash/ram?) that collects these messages and maybe a menu-entry to view the lasted XXX messages
neutrino.cpp:3111:2: warning: #warning TODO: count restart-rate to catch endless loops
neutrino.cpp: In member function `virtual int CNeutrinoApp::exec(CMenuTarget*, const std::string&)':
neutrino.cpp:4006: error: `CFileBrowser' was not declared in this scope
neutrino.cpp:4006: error: expected `;' before "b"
neutrino.cpp:4007: error: `b' was not declared in this scope
neutrino.cpp:4006: warning: unused variable 'CFileBrowser'

patch:

Code: Alles auswählen

Index: apps/tuxbox/neutrino/src/neutrino.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/neutrino.cpp,v
retrieving revision 1.1046
diff -u -b -B -r1.1046 neutrino.cpp
--- a/apps/tuxbox/neutrino/src/neutrino.cpp	7 Sep 2010 09:42:55 -0000	1.1046
+++ b/apps/tuxbox/neutrino/src/neutrino.cpp	4 Oct 2010 07:46:07 -0000
@@ -65,6 +65,7 @@
 
 #include "global.h"
 #include "neutrino.h"
+#include "gui/filebrowser.h"
 #ifndef HAVE_TRIPLEDRAGON
 #include <dbox/fp.h>
 #endif

[GetAway]

Kann nicht sein, Standard-CVS baut durch. Welche Features hast du denn disabled?

[dbt]

Da ist höchstens was zu viel, kommetiere das mal aus:

Code: Alles auswählen

	else if(actionKey == "epgdir")
	{
		parent->hide();
		CFileBrowser b;
		b.Dir_Mode=true;
		if (b.exec(g_settings.epg_dir.c_str()))
		{
			if((b.getSelectedFile()->Name) == "/")
			{
				// if selected dir is root -> clear epg_dir
				g_settings.epg_dir = "";
			} else {
				g_settings.epg_dir = b.getSelectedFile()->Name + "/";
			}
			SendSectionsdConfig(); // update notifier
		}
		return menu_return::RETURN_REPAINT;
	}

Das wird nicht gebraucht, zumindest nicht an der Stelle, hatte ich wahrscheinlich neulich vergessen rauszunehmen, allerdings scheint trotzdem bei dir irgendwas anders zu sein, sonst würde das bei mehreren nicht bauen. Ich tippe mal auf ein Makeproblem.

[idl0r]

Kann nicht sein, Standard-CVS baut durch. Welche Features hast du denn disabled?

./configure --with-cvsdir="/home/idl0r/tuxbox" --prefix="/home/idl0r/tuxbox/dbox2" --enable-maintainer-mode --enable-german-keymaps --with-boxtype=dbox2 --with-procps --with-ucodesdir="/home/idl0r/tuxbox/idl0r/ucodes/" --with-curlversion=current --with-freetypeversion=current --with-customizationsdir="/home/idl0r/tuxbox/idl0r/custom" --disable-audioplayer --disable-movieplayer --disable-pictureviewer --enable-aformat --enable-cdkVcInfo --enable-dropbear --enable-eraseall --enable-input --enable-lirc --enable-kb2rcd --enable-openntpd --enable-strace --enable-sysinfo --disable-tuxcom --disable-tuxmail --disable-enigma-movieplayer --disable-enigma-rss

Da ist höchstens was zu viel, kommetiere das mal aus:

Code: Alles auswählen

	else if(actionKey == "epgdir")
	{
		parent->hide();
		CFileBrowser b;
		b.Dir_Mode=true;
		if (b.exec(g_settings.epg_dir.c_str()))
		{
			if((b.getSelectedFile()->Name) == "/")
			{
				// if selected dir is root -> clear epg_dir
				g_settings.epg_dir = "";
			} else {
				g_settings.epg_dir = b.getSelectedFile()->Name + "/";
			}
			SendSectionsdConfig(); // update notifier
		}
		return menu_return::RETURN_REPAINT;
	}

Das wird nicht gebraucht, zumindest nicht an der Stelle, hatte ich wahrscheinlich neulich vergessen rauszunehmen, allerdings scheint trotzdem bei dir irgendwas anders zu sein, sonst würde das bei mehreren nicht bauen. Ich tippe mal auf ein Makeproblem.

Das auskommentieren aendert auch nichts an der tatsache das CFileBrowser da benutzt wird aber der filebrowser.h header nicht included wurde/wird.

filebrowser.h wird in folgenden headern included:

src/gui/pictureviewer.h:#include "gui/filebrowser.h"
src/gui/epgplus.h:#include "filebrowser.h"
src/gui/movieplayer.h:#include "gui/filebrowser.h"
src/gui/audioplayer.h:#include "gui/filebrowser.h"
src/gui/upnpbrowser.h:#include "gui/filebrowser.h"

Weder in neutrino.cpp noch in neutrino.h wird der header included obwohl er gebraucht wird, somit ist das meines erachtens ein bug im source.

*edit
achso.. du meinst das ganze statement.. hm

*nochmal edit
Ich sehe gerade das dass gesamte statement nun in gui/miscsettings_menu.cpp ist, hast also recht, weg mit dem statement.

[Gaucho316]

filebrowser.h wird in folgenden headern included:
src/gui/pictureviewer.h:#include "gui/filebrowser.h"
src/gui/epgplus.h:#include "filebrowser.h"
src/gui/movieplayer.h:#include "gui/filebrowser.h"
src/gui/audioplayer.h:#include "gui/filebrowser.h"
src/gui/upnpbrowser.h:#include "gui/filebrowser.h"

Dann müsste man filebrowser.h wirklich noch in neutrino.cpp includieren. Wenn man nämlich mit
--disable-pictureviewer
--disable-epgplus
--disable-movieplayer
--disable-audioplayer
baut, kann doch FILEBROWSER_NUMBER_OF_SORT_VARIANTS nicht gefunden werden. Oder sehe ich das falsch? Ich hab's allerdings nicht getestet.

Außerdem sehe ich gerade, dass in moviebrowser.cpp auch

Code: Alles auswählen

#define FILEBROWSER_NUMBER_OF_SORT_VARIANTS 5

steht. Das ist wohl überflüssig, da auch hier filebrowser.h includiert und somit FILEBROWSER_NUMBER_OF_SORT_VARIANTS zweimal definiert wird.

[GetAway]

@dbt

Ich schieb mal. Du checkst ja gerade wieder ein.

[dbt]

Sollte erledigt sein. Bitte mal testen. Wegen der "disables" musste der header doch wieder rein. @Gaucho316, danke für den Hinweis!