softhddevice unter vdr 2.1.10

MegaV0lt · 11. Februar 2015

Da kommen massig Warnungen und Fehle:

Code

*** Plugin softhddevice:
g++ -O3 -Wall -march=native -O3 -pipe -fPIC -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE  -I/usr/local/src/vdr-2.1.10/include -I/usr/include/alsa  	-DPLUGIN_NAME_I18N='"softhddevice"' -D_GNU_SOURCE  -DAV_INFO -DAV_INFO_TIME=3000	 -DUSE_PIP			 -DUSE_VDR_SPU			 -DUSE_ALSA -DUSE_OSS -DUSE_VDPAU -DUSE_SCREENSAVER -DUSE_SWRESAMPLE  -DGIT_REV='"2ceeb6d"' -I/usr/local/src/vdr-2.1.10/include  -g -W -Wall -Wextra -Winit-self -Werror=overloaded-virtual   -c -o softhddevice.o softhddevice.cpp
cc -O3 -Wall -march=native -O3 -pipe -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE  -I/usr/local/src/vdr-2.1.10/include -I/usr/include/alsa  	-DPLUGIN_NAME_I18N='"softhddevice"' -D_GNU_SOURCE  -DAV_INFO -DAV_INFO_TIME=3000	 -DUSE_PIP			 -DUSE_VDR_SPU			 -DUSE_ALSA -DUSE_OSS -DUSE_VDPAU -DUSE_SCREENSAVER -DUSE_SWRESAMPLE  -DGIT_REV='"2ceeb6d"' -I/usr/local/src/vdr-2.1.10/include  -g -W -Wall -Wextra -Winit-self -Wdeclaration-after-statement   -c -o softhddev.o softhddev.c
In file included from /usr/local/src/vdr-2.1.10/include/vdr/i18n.h:14:0,
             	from /usr/local/src/vdr-2.1.10/include/vdr/config.h:19,
             	from /usr/local/src/vdr-2.1.10/include/vdr/interface.h:13,
             	from softhddevice.cpp:25:
/usr/local/src/vdr-2.1.10/include/vdr/tools.h:460:15: warning: unused parameter â€˜ListObjectâ€™ [-Wunused-parameter]
In file included from /usr/local/src/vdr-2.1.10/include/vdr/config.h:20:0,
             	from /usr/local/src/vdr-2.1.10/include/vdr/interface.h:13,
             	from softhddevice.cpp:25:
/usr/local/src/vdr-2.1.10/include/vdr/font.h:53:7: warning: unused parameter â€˜sâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/font.h:58:16: warning: unused parameter â€˜Pixmapâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/font.h:58:16: warning: unused parameter â€˜xâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/font.h:58:16: warning: unused parameter â€˜yâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/font.h:58:16: warning: unused parameter â€˜sâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/font.h:58:16: warning: unused parameter â€˜ColorFgâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/font.h:58:16: warning: unused parameter â€˜ColorBgâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/font.h:58:16: warning: unused parameter â€˜Widthâ€™ [-Wunused-parameter]
cc -O3 -Wall -march=native -O3 -pipe -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE  -I/usr/local/src/vdr-2.1.10/include -I/usr/include/alsa  	-DPLUGIN_NAME_I18N='"softhddevice"' -D_GNU_SOURCE  -DAV_INFO -DAV_INFO_TIME=3000	 -DUSE_PIP			 -DUSE_VDR_SPU			 -DUSE_ALSA -DUSE_OSS -DUSE_VDPAU -DUSE_SCREENSAVER -DUSE_SWRESAMPLE  -DGIT_REV='"2ceeb6d"' -I/usr/local/src/vdr-2.1.10/include  -g -W -Wall -Wextra -Winit-self -Wdeclaration-after-statement   -c -o video.o video.c
In file included from /usr/local/src/vdr-2.1.10/include/vdr/skins.h:14:0,
             	from /usr/local/src/vdr-2.1.10/include/vdr/interface.h:15,
             	from softhddevice.cpp:25:
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:236:16: warning: unused parameter â€˜Channelâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:241:16: warning: unused parameter â€˜Scheduleâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:241:16: warning: unused parameter â€˜EitEventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:241:16: warning: unused parameter â€˜TableIDâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:241:16: warning: unused parameter â€˜Versionâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:246:16: warning: unused parameter â€˜Channelâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:252:16: warning: unused parameter â€˜EventIDâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:252:16: warning: unused parameter â€˜StartTimeâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:252:16: warning: unused parameter â€˜TableIDâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:252:16: warning: unused parameter â€˜Versionâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:256:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:256:16: warning: unused parameter â€˜EventIDâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:257:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:257:16: warning: unused parameter â€˜Titleâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:258:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:258:16: warning: unused parameter â€˜ShortTextâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:259:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:259:16: warning: unused parameter â€˜Descriptionâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:260:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:260:16: warning: unused parameter â€˜Contentsâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:261:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:261:16: warning: unused parameter â€˜ParentalRatingâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:262:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:262:16: warning: unused parameter â€˜StartTimeâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:263:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:263:16: warning: unused parameter â€˜Durationâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:264:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:264:16: warning: unused parameter â€˜Vpsâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:265:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:265:16: warning: unused parameter â€˜Componentsâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:266:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:268:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:271:16: warning: unused parameter â€˜Scheduleâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:273:16: warning: unused parameter â€˜Scheduleâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:273:16: warning: unused parameter â€˜SegmentStartâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:273:16: warning: unused parameter â€˜SegmentEndâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:273:16: warning: unused parameter â€˜TableIDâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:273:16: warning: unused parameter â€˜Versionâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:276:16: warning: unused parameter â€˜Channelâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:276:16: warning: unused parameter â€˜OnlyRunningStatusâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:279:16: warning: unused parameter â€˜Modifiedâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/epg.h:279:16: warning: unused parameter â€˜OnlyRunningStatusâ€™ [-Wunused-parameter]
In file included from /usr/local/src/vdr-2.1.10/include/vdr/skins.h:17:0,
             	from /usr/local/src/vdr-2.1.10/include/vdr/interface.h:15,
             	from softhddevice.cpp:25:
/usr/local/src/vdr-2.1.10/include/vdr/positioner.h:113:16: warning: unused parameter â€˜Directionâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/positioner.h:116:16: warning: unused parameter â€˜Directionâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/positioner.h:116:16: warning: unused parameter â€˜Stepsâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/positioner.h:124:16: warning: unused parameter â€˜Directionâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/positioner.h:131:16: warning: unused parameter â€˜Numberâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/positioner.h:135:16: warning: unused parameter â€˜Numberâ€™ [-Wunused-parameter]
In file included from /usr/local/src/vdr-2.1.10/include/vdr/interface.h:15:0,
             	from softhddevice.cpp:25:
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:40:16: warning: unused parameter â€˜Redâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:40:16: warning: unused parameter â€˜Greenâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:40:16: warning: unused parameter â€˜Yellowâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:40:16: warning: unused parameter â€˜Blueâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:43:16: warning: unused parameter â€˜Typeâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:43:16: warning: unused parameter â€˜Textâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:178:16: warning: unused parameter â€˜MenuSortModeâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:214:16: warning: unused parameter â€˜Eventâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:214:16: warning: unused parameter â€˜Indexâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:214:16: warning: unused parameter â€˜Currentâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:214:16: warning: unused parameter â€˜Selectableâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:214:16: warning: unused parameter â€˜Channelâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:214:16: warning: unused parameter â€˜WithDateâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:214:16: warning: unused parameter â€˜TimerMatchâ€™ [-Wunused-parameter]
In file included from /usr/local/src/vdr-2.1.10/include/vdr/interface.h:15:0,
             	from softhddevice.cpp:25:
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:224:16: warning: unused parameter â€˜Timerâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:224:16: warning: unused parameter â€˜Indexâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:224:16: warning: unused parameter â€˜Currentâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:224:16: warning: unused parameter â€˜Selectableâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:231:16: warning: unused parameter â€˜Channelâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:231:16: warning: unused parameter â€˜Indexâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:231:16: warning: unused parameter â€˜Currentâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:231:16: warning: unused parameter â€˜Selectableâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:231:16: warning: unused parameter â€˜WithProviderâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:240:16: warning: unused parameter â€˜Recordingâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:240:16: warning: unused parameter â€˜Indexâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:240:16: warning: unused parameter â€˜Currentâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:240:16: warning: unused parameter â€˜Selectableâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:240:16: warning: unused parameter â€˜Levelâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:240:16: warning: unused parameter â€˜Totalâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/skins.h:240:16: warning: unused parameter â€˜Newâ€™ [-Wunused-parameter]
In file included from /usr/local/src/vdr-2.1.10/include/vdr/device.h:14:0,
             	from /usr/local/src/vdr-2.1.10/include/vdr/dvbdevice.h:15,
             	from /usr/local/src/vdr-2.1.10/include/vdr/menuitems.h:14,
             	from /usr/local/src/vdr-2.1.10/include/vdr/plugin.h:14,
             	from softhddevice.cpp:26:
/usr/local/src/vdr-2.1.10/include/vdr/ci.h:94:15: warning: unused parameter â€˜Bufferâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/ci.h:94:15: warning: unused parameter â€˜MaxLengthâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/ci.h:99:16: warning: unused parameter â€˜Bufferâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/ci.h:99:16: warning: unused parameter â€˜Lengthâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/ci.h:101:16: warning: unused parameter â€˜Slotâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/ci.h:104:25: warning: unused parameter â€˜Slotâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/ci.h:106:16: warning: unused parameter â€˜Deviceâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/ci.h:106:16: warning: unused parameter â€˜Queryâ€™ [-Wunused-parameter]
In file included from /usr/local/src/vdr-2.1.10/include/vdr/dvbdevice.h:15:0,
             	from /usr/local/src/vdr-2.1.10/include/vdr/menuitems.h:14,
             	from /usr/local/src/vdr-2.1.10/include/vdr/plugin.h:14,
             	from softhddevice.cpp:26:
/usr/local/src/vdr-2.1.10/include/vdr/device.h:681:17: warning: unused parameter â€˜Rectâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/device.h:681:17: warning: unused parameter â€˜Alignmentâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/device.h:702:16: warning: unused parameter â€˜Rectâ€™ [-Wunused-parameter]
In file included from /usr/local/src/vdr-2.1.10/include/vdr/menuitems.h:15:0,
             	from /usr/local/src/vdr-2.1.10/include/vdr/plugin.h:14,
             	from softhddevice.cpp:26:
/usr/local/src/vdr-2.1.10/include/vdr/osdbase.h:83:20: warning: unused parameter â€˜Keyâ€™ [-Wunused-parameter]
In file included from softhddevice.cpp:27:0:
/usr/local/src/vdr-2.1.10/include/vdr/player.h:39:16: warning: unused parameter â€˜Onâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:57:16: warning: unused parameter â€˜Currentâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:57:16: warning: unused parameter â€˜Totalâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:57:16: warning: unused parameter â€˜SnapToIFrameâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:60:16: warning: unused parameter â€˜Playâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:60:16: warning: unused parameter â€˜Forwardâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:60:16: warning: unused parameter â€˜Speedâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:66:16: warning: unused parameter â€˜Typeâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:66:16: warning: unused parameter â€˜TrackIdâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:70:16: warning: unused parameter â€˜Typeâ€™ [-Wunused-parameter]
/usr/local/src/vdr-2.1.10/include/vdr/player.h:70:16: warning: unused parameter â€˜TrackIdâ€™ [-Wunused-parameter]
softhddevice.cpp: In member function â€˜virtual void cSoftOsd::Flush()â€™:
softhddevice.cpp:499:32: error: invalid conversion from â€˜cPixmap*â€™ to â€˜cPixmapMemory*â€™ [-fpermissive]
In file included from softhddevice.cpp:1380:0:
/usr/local/src/vdr-2.1.10/include/vdr/receiver.h: At global scope:
/usr/local/src/vdr-2.1.10/include/vdr/receiver.h:29:16: warning: unused parameter â€˜Onâ€™ [-Wunused-parameter]
softhddevice.cpp: In member function â€˜virtual void cSoftHdDevice::TrickSpeed(int, bool)â€™:
softhddevice.cpp:2291:5: warning: too many arguments for format [-Wformat-extra-args]
make[1]: *** [softhddevice.o] Error 1
make[1]: *** Waiting for unfinished jobs....
video.c: In function â€˜VideoSetPtsâ€™:
video.c:450:28: warning: unused parameter â€˜video_ctxâ€™ [-Wunused-parameter]
make[1]: *** wait: No child processes.  Stop.
ls: cannot access /usr/local/src/vdr-2.1.10/PLUGINS/src/softhddevice/po/*.mo: No such file or directory

Alles anzeigen

Mod.: Bitte bei langen Listings Spoiler verwenden, das verbessert die Lesbarkeit ungemein.

jsffm · 11. Februar 2015

Ich stehe vor dem gleichen Problem

johns · 11. Februar 2015

Changelog lesen:

Zitat

- cOsd::RenderPixmaps() now returns a pointer to cPixmap instead of cPixmapMemory
(suggested by Thomas Reufer). This is necessary to allow plugins with derived
cPixmap implementations to use this function. Plugins that use this function
with cPixmapMemory now need to add a dynamic cast to the call, as in

cPixmapMemory *pm = dynamic_cast<cPixmapMemory *>(RenderPixmaps()));

They also need to call DestroyPixmap(pm) instead of "delete pm" to properly release
the resulting pixmap after use.

Alles anzeigen

Viel Spass beim fixen,
Johns

kls · 11. Februar 2015

Die Warnungen können alle ignoriert werden (gab's die in VDR 2.1.9 nicht?).
Entscheidend ist nur das hier:

softhddevice.cpp: In member function â€˜virtual void cSoftOsd::Flush()â€™:
softhddevice.cpp:499:32: error: invalid conversion from â€˜cPixmap*â€™ to â€˜cPixmapMemory*â€™ [-fpermissive]

Hierzu hatte ich in der Freigabenotiz geschrieben:

Code

- cOsd::RenderPixmaps() now returns a pointer to cPixmap instead of cPixmapMemory
  (suggested by Thomas Reufer). This is necessary to allow plugins with derived
  cPixmap implementations to use this function. Plugins that use this function
  with cPixmapMemory now need to add a dynamic cast to the call, as in


    cPixmapMemory *pm = dynamic_cast<cPixmapMemory *>(RenderPixmaps()));


  They also need to call DestroyPixmap(pm) instead of "delete pm" to properly release
  the resulting pixmap after use.
  The dvbhddevice plugin has been modified accordingly.

Alles anzeigen

Ihr müsst also den o.g. Typecast in softhddevice.cpp einbauen, und das weiter unten befindliche "delete" durch ein DestroyPixmap() ersetzen.

Klaus

Edit: johns war schneller

seahawk1986 · 11. Februar 2015

Zitat von kls

Ihr müsst also den o.g. Typecast in softhddevice.cpp einbauen, und das weiter unten befindliche "delete" durch ein DestroyPixmap() ersetzen.

Also so? Damit baut es zumindest:

Diff

diff --git a/softhddevice.cpp b/softhddevice.cpp
index b3467a5..a744052 100644
--- a/softhddevice.cpp
+++ b/softhddevice.cpp
@@ -496,7 +496,7 @@ void cSoftOsd::Flush(void)
     }
 
     LOCK_PIXMAPS;
-    while ((pm = RenderPixmaps())) {
+    while ((pm = dynamic_cast<cPixmapMemory *>(RenderPixmaps()))) {
        int x;
        int y;
        int w;
@@ -513,7 +513,7 @@ void cSoftOsd::Flush(void)
 #endif
        OsdDrawARGB(x, y, w, h, pm->Data());
 
-       delete pm;
+       DestroyPixmap(pm);
     }
     Dirty = 0;
 }

Alles anzeigen

kls · 11. Februar 2015

Ja, so sollte es gehen.
Das dvbhddevice (TT S2-6400) funktioniert damit einwandfrei.

Klaus

johns · 11. Februar 2015

Zitat von seahawk1986

Also so? Damit baut es zumindest:

Diff

diff --git a/softhddevice.cpp b/softhddevice.cpp
index b3467a5..a744052 100644
--- a/softhddevice.cpp
+++ b/softhddevice.cpp
@@ -496,7 +496,7 @@ void cSoftOsd::Flush(void)
     }
 
     LOCK_PIXMAPS;
-    while ((pm = RenderPixmaps())) {
+    while ((pm = dynamic_cast<cPixmapMemory *>(RenderPixmaps()))) {
        int x;
        int y;
        int w;
@@ -513,7 +513,7 @@ void cSoftOsd::Flush(void)
 #endif
        OsdDrawARGB(x, y, w, h, pm->Data());
 
-       delete pm;
+       DestroyPixmap(pm);
     }
     Dirty = 0;
 }

Alles anzeigen

Etwas komisch liest es sich schon. Da muß ich mir erst die Sourcen und Headerfiles vom VDR ansehen.
Man holt pixmap casted die nach pixmapmemory und gibt dann pixmap frei. *schüttel*

Johns

jsffm · 11. Februar 2015

Zumindest läuft es damit. Ich musste allerdings den Patch von Hand einbauen, warum auch immer.

Copperhead · 11. Februar 2015

Weil Patches aus Codetags nicht funktionieren. Die untere leere Zeile wird immer unterschlagen.

kls · 11. Februar 2015

Zitat von johns

Etwas komisch liest es sich schon. Da muß ich mir erst die Sourcen und Headerfiles vom VDR ansehen.
Man holt pixmap casted die nach pixmapmemory und gibt dann pixmap frei. *schüttel*

Ein Plugin, welches ein OSD realisiert, kann eine abgeleitete cPixmap-Klasse implementieren, um z.B. eine GPU für das Zeichnen zu verwenden. Um die Funktion cOsd::RenderPixmaps() verwenden zu können muß daher deren Rückgabewert cPixmap* sein und nicht cPixmapMemory*. Das war genau genommen ein Design-Fehler meinerseits damals, ich hatte zu sehr an die konkrete Default-Implementierung mit cPixmapMemory gedacht. Draufgekommen ist Thomas Reufer beim Einbau der GPU-Unterstützung für Pixmaps im rpihddevice-Plugin - zum Glück gerade noch rechtzeitig vor der Freigabe der 2.2.0.

Da der Typecast ja innerhalb des Codes gemacht wird, der "weiß" um welche konkrete Art von Pixmap es sich handelt, ist das auch sicher. Und daß cOsd::DestroyPixmap mit dem allgemeinen cPixmap* arbeitet ist auch ganz normal.

Klaus

johns · 11. Februar 2015

Zitat von jsffm

Zumindest läuft es damit. Ich musste allerdings den Patch von Hand einbauen, warum auch immer.

Kann man es so für 2.x.x verwenden oder braucht es #ifdef's.

Johns

kls · 11. Februar 2015

Sollte ohne ifdefs auch mit älteren Versionen funktionieren.
cOsd::DestroyPixmap() gab es auch da schon.

Klaus

johns · 12. Februar 2015

Änderung ist nun im GIT.

Johns

MegaV0lt · 12. Februar 2015

Vielen Dank. Bauprozess läuft...

jsffm · 12. Februar 2015

Compiliert auch unter 2.0.7

LordJaxom · 12. Februar 2015

Wenn man weiß, dass es sich um ein cPixmapMemory-Objekt handelt, reicht auch ein static_cast. Der dynamic_cast führt zusätzlich Runtime-Typechecks durch und gibt nullptr zurück, wenn es sich nicht um ein solches handelt. Ist nicht tragisch, verbrät aber potentiell ein klitzekleines bisschen Leistung, die nicht verbraten werden müsste.

mini73 · 12. Februar 2015

Ich denke, der Leistungsschwund ist für das Mehr an Sicherheit zu vernachlässigen.

Lars.

johns · 16. Februar 2015

Mich stört mehr dieser Report: http://projects.vdr-developer.org/issues/2100
Kann ich nachvollziehen mit default Skin und fast keinen Plugins. Edit meine Version ist 2.1.8-r2.

Pointer schaut ok aus.

Johns

fnu · 16. Februar 2015

johns

Zitat von scop

My vdr is version 2.0.7.

Die API Änderung gilt doch erst ab VDR 2.1.10, die am Donnerstag vorr. VDR 2.2.0 (stable) wird ... ?

Regards
fnu

mini73 · 16. Februar 2015

Die Code-Änderung kann man aber auch mit vdr 2.0.x verwenden, das macht da keinen Unterschied.

Lars.

softhddevice unter vdr 2.1.10

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema