ldo / dvdauthor

continue development of DVD-authoring tool
http://dvdauthor.sourceforge.net/
GNU General Public License v2.0
71 stars 20 forks source link

'up' attribute not supported in pgc #13

Closed lu-zero closed 10 years ago

lu-zero commented 10 years ago
<pgc entry="subtitle" up="1">
ldo commented 10 years ago

What should it do?

lu-zero commented 10 years ago

I can see there is an 'up', 'next', and 'prev' attributes

          /* add pgc nav info */
            if (pgc->goup_pgc_nr)
              {
                AddAttribute(pgcNode, "up", "%d", pgc->goup_pgc_nr);
              } /*if*/
            if (pgc->next_pgc_nr)
              {
                AddAttribute(pgcNode, "next", "%d", pgc->next_pgc_nr);
              } /*if*/
            if (pgc->prev_pgc_nr)
              {
                AddAttribute(pgcNode, "prev", "%d", pgc->prev_pgc_nr);
              } /*if*/

but simply grepping to check where those are set returns nothing (I couldn't see how that struct is populated)

ldo commented 10 years ago

So what is it supposed to do?

lu-zero commented 10 years ago

spent a little time to read dvdread and dvdnav

We have those 3 values

  B2N_16(pgc->next_pgc_nr);
  B2N_16(pgc->prev_pgc_nr);
  B2N_16(pgc->goup_pgc_nr);

And apparently they can be used to:

ldo commented 10 years ago

Sutmit a patch, then.