Bugzilla Bug 117386

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.