<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "http://bugs.gentoo.org/bugzilla.dtd">

<bugzilla version="2.22.7"
          urlbase="http://bugs.gentoo.org/"
          maintainer="bugzilla@gentoo.org"
>

    <bug>
          <bug_id>117386</bug_id>
          
          <creation_ts>2006-01-01 13:22 0000</creation_ts>
          <short_desc>k9copy ebuild failure k9main.cpp</short_desc>
          <delta_ts>2006-01-06 18:19:36 0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>Gentoo Linux</product>
          <component>Ebuilds</component>
          <version>unspecified</version>
          <rep_platform>All</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>mystified@satx.rr.com</reporter>
          <assigned_to>media-optical@gentoo.org</assigned_to>
          <cc>geoff@electron.me.uk</cc>

      

      
          <long_desc isprivate="0">
            <who>mystified@satx.rr.com</who>
            <bug_when>2006-01-01 13:22:10 0000</bug_when>
            <thetext>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)&apos;:
../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&amp;)&apos;:
k9main.cpp:56: warning: unused parameter `const QStringList&amp;sl&apos;
k9main.cpp: In constructor `k9DVDListItem::k9DVDListItem(QObject*, ckLvItem*,
   eStreamType)&apos;:
k9main.cpp:103: warning: enumeration value `NONE&apos; not handled in switch
k9main.cpp: In constructor `k9DVDListItem::k9DVDListItem(QObject*, ckLvItem*,
   eStreamType)&apos;:
k9main.cpp:103: warning: enumeration value `NONE&apos; not handled in switch
k9main.cpp: In member function `virtual void ckLvItem::stateChange(bool)&apos;:
k9main.cpp:135: warning: passing `float&apos; for converting 1 of `void
   KSelector::setValue(int)&apos;
k9main.cpp: In member function `virtual void ckLvLangItem::stateChange(bool)&apos;:
k9main.cpp:145: warning: passing `float&apos; for converting 1 of `void
   KSelector::setValue(int)&apos;
k9main.cpp: In member function `void k9Main::slot_progress(QString)&apos;:
k9main.cpp:264: warning: unused parameter `QString str&apos;
k9main.cpp: In member function `void k9Main::updateSelection()&apos;:
k9main.cpp:351: warning: comparison between signed and unsigned integer
   expressions
k9main.cpp:369: warning: enumeration value `NONE&apos; not handled in switch
k9main.cpp: In member function `void k9Main::checkLang(QString, eStreamType,
   bool)&apos;:
k9main.cpp:399: warning: enumeration value `VID&apos; not handled in switch
k9main.cpp:399: warning: enumeration value `NONE&apos; not handled in switch
k9main.cpp: In member function `void k9Main::checkTitle(bool, ckLvItem*)&apos;:
k9main.cpp:437: warning: enumeration value `VID&apos; not handled in switch
k9main.cpp:437: warning: enumeration value `NONE&apos; not handled in switch
k9main.cpp: In member function `void k9Main::updateLvLang(eStreamType, const
   QString&amp;)&apos;:
k9main.cpp:471: warning: enumeration value `VID&apos; not handled in switch
k9main.cpp:471: warning: enumeration value `NONE&apos; not handled in switch
k9main.cpp: In member function `void k9Main::checkAll(bool)&apos;:
k9main.cpp:521: warning: enumeration value `NONE&apos; not handled in switch
k9main.cpp: In member function `void k9Main::itemRenamed(QListViewItem*, int)&apos;:
k9main.cpp:536: warning: unused parameter `int col&apos;
k9main.cpp: In member function `virtual void k9Main::ckMenuClick()&apos;:
k9main.cpp:736: warning: passing `float&apos; for converting 1 of `void
   KSelector::setValue(int)&apos;
k9main.cpp: In member function `void k9Main::fillLvLanguages()&apos;:
k9main.cpp:768: error: jump to case label
k9main.cpp:764: error:   crosses initialization of `k9DVDSubtitle*l_sub&apos;
k9main.cpp:773: warning: enumeration value `VID&apos; not handled in switch
k9main.cpp:773: warning: enumeration value `NONE&apos; 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&apos;
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/var/tmp/portage/k9copy-1.0.2/work/k9copy-1.0.2&apos;
make: *** [all] Error 2</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>rickfharris@yahoo.com.au</who>
            <bug_when>2006-01-01 13:33:24 0000</bug_when>
            <thetext>What version of k9copy ?

What version of QT ?

`emerge info` ?</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>rickfharris@yahoo.com.au</who>
            <bug_when>2006-01-01 13:34:19 0000</bug_when>
            <thetext>Sorry, so very tired, I see k9copy-1.0.2</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>geoff@electron.me.uk</who>
            <bug_when>2006-01-02 11:31:31 0000</bug_when>
            <thetext>Created an attachment (id=76010)
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.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>metalgod@gentoo.org</who>
            <bug_when>2006-01-02 19:01:30 0000</bug_when>
            <thetext>Rick can you test with that patch please ?</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>rickfharris@yahoo.com.au</who>
            <bug_when>2006-01-03 12:51:58 0000</bug_when>
            <thetext>I didn&apos;t get the bug originally, but can confirm that k9copy still works here with the patch applied.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>geoff@electron.me.uk</who>
            <bug_when>2006-01-05 02:54:13 0000</bug_when>
            <thetext>Maintainer reports that patch in now included in upstream CVS.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>metalgod@gentoo.org</who>
            <bug_when>2006-01-06 18:19:36 0000</bug_when>
            <thetext>patch commited Thanks for the patch!

Feel free to reopen the bug if this issue still exists.</thetext>
          </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>76010</attachid>
            <date>2006-01-02 11:31 0000</date>
            <desc>Proposed patch for compile problem</desc>
            <filename>k9copy-switch.patch</filename>
            <type>text/plain</type>
            <data encoding="base64">LS0tIGs5Y29weS0xLjAuMi1vbGQvc3JjL2s5bWFpbi5jcHAJMjAwNS0xMi0xOCAwODoxMToxOS4w
MDAwMDAwMDAgKzAwMDAKKysrIGs5Y29weS0xLjAuMi9zcmMvazltYWluLmNwcAkyMDA2LTAxLTAy
IDE5OjA1OjUxLjAwMDAwMDAwMCArMDAwMApAQCAtNzUyLDYgKzc1Miw5IEBACiAKIAogdm9pZCBr
OU1haW46OmZpbGxMdkxhbmd1YWdlcygpIHsKKyAgICBrOURWRFN1YnRpdGxlICpsX3N1YjsKKyAg
ICBrOURWREF1ZGlvU3RyZWFtICpsX2F1ZHM7CisKICAgICBsdkxhbmd1YWdlcy0+Y2xlYXIoKTsK
ICAgICBsYW5nQXVkSXRlbXMuY2xlYXIoKTsKICAgICBsYW5nU3ViSXRlbXMuY2xlYXIoKTsKQEAg
LTc2MSwxMiArNzY0LDEyIEBACiAgICAgICAgICAgICBrOURWRExpc3RJdGVtICpsaXRlbT0oazlE
VkRMaXN0SXRlbSopaXRlbXMuYXQoaSk7CiAgICAgICAgICAgICBzd2l0Y2ggKGxpdGVtLT5zdHJl
YW1UeXBlKSB7CiAgICAgICAgICAgICBjYXNlIFNVQjoKLSAgICAgICAgICAgICAgICBrOURWRFN1
YnRpdGxlICpsX3N1Yj1saXRlbS0+c3VidGl0bGU7CisgICAgICAgICAgICAgICAgbF9zdWI9bGl0
ZW0tPnN1YnRpdGxlOwogICAgICAgICAgICAgICAgIGlmIChzbFN1YkxhbmcuY29udGFpbnMobF9z
dWItPmdldGxhbmd1YWdlKCkpPT0wKQogICAgICAgICAgICAgICAgICAgICBzbFN1YkxhbmcuYXBw
ZW5kKGxfc3ViLT5nZXRsYW5ndWFnZSgpKTsKICAgICAgICAgICAgICAgICBicmVhazsKICAgICAg
ICAgICAgIGNhc2UgQVVEOgotICAgICAgICAgICAgICAgIGs5RFZEQXVkaW9TdHJlYW0gKmxfYXVk
cz1saXRlbS0+YXVkaW9TdHJlYW07CisgICAgICAgICAgICAgICAgbF9hdWRzPWxpdGVtLT5hdWRp
b1N0cmVhbTsKICAgICAgICAgICAgICAgICBpZiAoc2xBdWRMYW5nLmNvbnRhaW5zKGxfYXVkcy0+
Z2V0bGFuZ3VhZ2UoKSk9PTApCiAgICAgICAgICAgICAgICAgICAgIHNsQXVkTGFuZy5hcHBlbmQo
bF9hdWRzLT5nZXRsYW5ndWFnZSgpKTsKICAgICAgICAgICAgICAgICBicmVhazsK
</data>        

          </attachment>
    </bug>

</bugzilla>