Re: [QDVDAuthor-devel] A few remarks on 0.0.6
Brought to you by:
qdvdauthor
From: qdvdauthor <qdv...@us...> - 2004-06-23 13:53:25
|
Steven, see below for the answers Riviere Informatique SARL wrote: > Hi, > > Great program! > There's a few bugs/minor problems I've noticed. > Since I only subscribed to this list a short while ago, I'm not sure > if they were mentioned before - in which case I'm sure I'll hear from > you :-) > I've browsed the archive, but I've got a very slow connection (No ADSL > available where I live )-: and I have not read everything. > > Anyway, here goes: > 1 - After adding a text to a menu the 'Edit/Delete/Use as Button' > submenu isn't immediately available. It becomes available after adding > a second text. In that case it's available right away for both texts. > Seems a bit weird I will check into this. > 2 - When creating a DVD the 'arecord' option I get doesn't use " > around the filename, which causes a problem if you've got embedded > spaces. The offending line of code is in qdvdauthor.cpp on line 1320: > qsSound = QString ("arecord -f dat -twav -d 1 /dev/stdout | mp2enc -r > 48000 -o %1.mp2").arg(qsBackgroundFileName); Yep, I know about this and I fixed it some time ago. I am now using the Menu name only (without background). > > which I changed to > qsSound = QString ("arecord -f dat -twav -d 1 | mp2enc -r 48000 -o > *\"*%1.mp2*\"*").arg(qsBackgroundFileName); > > I've removed the /dev/stdout because that is the default for arecord > and I got a lot of errors (Buffer over or under flow - can't remember) > when it was present. Thank you for this hint, I will incorporate it ASAP Can I do the same change or the second arecord command ? > > 3 - The value of BACKGROUND_NAME as defined in global.h appears in the > above mentioned 'arecord' command line: > arecord -f dat -twav -d 1 | mp2enc -r 48000 -o "/tmp/Proj1/Main Menu > VMGMbackground.jpg.mp2" > but doesn't appear in the name of the input file on the next line: > /usr/local/bin/mplex -f 8 -o "/tmp/Proj1/Main Menu VMGM.mpg" > "/tmp/Proj1/Main Menu VMGM.m2v" "/tmp/Proj1/Main Menu VMGM.mp2" I fixed this by changing the arecord line ... > > 4 - It would be great if these 'default command strings' could be > modified in the setup or read from a config file This has cost me about a week but if you use the CVS version, you can do exactly this :) You can also choose your default command. > > 5 - When compiling I get this: > /usr/src/qdvdauthor-0.0.6/qdvdauthor/imagemagicklib.inc:95: warning: > the use of `tmpnam' is dangerous, better use `mkstemp' Yep I know, this is on the to do list fo version 12.1 ;) Just kidding, this was fixed by some one and I applied the patch. > > In fact, then man page of tmpnam sais the following, under Bugs: > Never use this function. Use tmpfile(3) instead. > > There's no easy conversion, since both these functions return a FILE > pointer rather than a string - and I don't know anything about QT yet, > so I can't help to rewrite the code. > > > I hope this helps to improve the program - it's a very good start > anyway - and certainly the best of the GUI's for dvdauthor I've seen > so far. > > > Regards, > Steven van de Beek > > > > ------------------------------------------------------- This SF.Net > email sponsored by Black Hat Briefings & Training. Attend Black Hat > Briefings & Training, Las Vegas July 24-29 - digital self defense, top > technical experts, no vendor pitches, unmatched networking > opportunities. Visit www.blackhat.com > _______________________________________________ QDVDAuthor-devel > mailing list QDV...@li... > https://lists.sourceforge.net/lists/listinfo/qdvdauthor-devel |