Bug 117386 - k9copy ebuild failure k9main.cpp
|
Bug#:
117386
|
Product: Gentoo Linux
|
Version: unspecified
|
Platform: All
|
|
OS/Version: Linux
|
Status: RESOLVED
|
Severity: normal
|
Priority: P2
|
|
Resolution: FIXED
|
Assigned To: media-optical@gentoo.org
|
Reported By: mystified@satx.rr.com
|
|
Component: Ebuilds
|
|
|
URL:
|
|
Summary: k9copy ebuild failure k9main.cpp
|
|
Keywords:
|
|
Status Whiteboard:
|
|
Opened: 2006-01-01 13:22 0000
|
In file included from kviewmpeg2.h:26,
from k9main.h:24,
from k9main.cpp:21:
../libk9copy/k9dvdtitle.h: In member function `virtual uchar
k9ChapterCell::setangleBlock(unsigned char)':
../libk9copy/k9dvdtitle.h:40: warning: no return statement in function
returning non-void
k9main.cpp: In member function `k9Main::k9Main(QWidget*, const char*, const
QStringList&)':
k9main.cpp:56: warning: unused parameter `const QStringList&sl'
k9main.cpp: In constructor `k9DVDListItem::k9DVDListItem(QObject*, ckLvItem*,
eStreamType)':
k9main.cpp:103: warning: enumeration value `NONE' not handled in switch
k9main.cpp: In constructor `k9DVDListItem::k9DVDListItem(QObject*, ckLvItem*,
eStreamType)':
k9main.cpp:103: warning: enumeration value `NONE' not handled in switch
k9main.cpp: In member function `virtual void ckLvItem::stateChange(bool)':
k9main.cpp:135: warning: passing `float' for converting 1 of `void
KSelector::setValue(int)'
k9main.cpp: In member function `virtual void ckLvLangItem::stateChange(bool)':
k9main.cpp:145: warning: passing `float' for converting 1 of `void
KSelector::setValue(int)'
k9main.cpp: In member function `void k9Main::slot_progress(QString)':
k9main.cpp:264: warning: unused parameter `QString str'
k9main.cpp: In member function `void k9Main::updateSelection()':
k9main.cpp:351: warning: comparison between signed and unsigned integer
expressions
k9main.cpp:369: warning: enumeration value `NONE' not handled in switch
k9main.cpp: In member function `void k9Main::checkLang(QString, eStreamType,
bool)':
k9main.cpp:399: warning: enumeration value `VID' not handled in switch
k9main.cpp:399: warning: enumeration value `NONE' not handled in switch
k9main.cpp: In member function `void k9Main::checkTitle(bool, ckLvItem*)':
k9main.cpp:437: warning: enumeration value `VID' not handled in switch
k9main.cpp:437: warning: enumeration value `NONE' not handled in switch
k9main.cpp: In member function `void k9Main::updateLvLang(eStreamType, const
QString&)':
k9main.cpp:471: warning: enumeration value `VID' not handled in switch
k9main.cpp:471: warning: enumeration value `NONE' not handled in switch
k9main.cpp: In member function `void k9Main::checkAll(bool)':
k9main.cpp:521: warning: enumeration value `NONE' not handled in switch
k9main.cpp: In member function `void k9Main::itemRenamed(QListViewItem*, int)':
k9main.cpp:536: warning: unused parameter `int col'
k9main.cpp: In member function `virtual void k9Main::ckMenuClick()':
k9main.cpp:736: warning: passing `float' for converting 1 of `void
KSelector::setValue(int)'
k9main.cpp: In member function `void k9Main::fillLvLanguages()':
k9main.cpp:768: error: jump to case label
k9main.cpp:764: error: crosses initialization of `k9DVDSubtitle*l_sub'
k9main.cpp:773: warning: enumeration value `VID' not handled in switch
k9main.cpp:773: warning: enumeration value `NONE' not handled in switch
make[2]: *** [k9main.o] Error 1
make[2]: Leaving directory
`/var/tmp/portage/k9copy-1.0.2/work/k9copy-1.0.2/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/var/tmp/portage/k9copy-1.0.2/work/k9copy-1.0.2'
make: *** [all] Error 2
What version of k9copy ?
What version of QT ?
`emerge info` ?
Sorry, so very tired, I see k9copy-1.0.2
Created an attachment (id=76010) [details]
Proposed patch for compile problem
The problem is caused by declaring variables within a switch statement, some
compilers do not like this. This patch should sort out the problem by
declaring the variables at the start of the function.
Rick can you test with that patch please ?
I didn't get the bug originally, but can confirm that k9copy still works here
with the patch applied.
Maintainer reports that patch in now included in upstream CVS.
patch commited Thanks for the patch!
Feel free to reopen the bug if this issue still exists.