search

HaikuArchives / ffmpegGUI

GUI for FFmpeg
MIT License
23 stars 10 forks source link

add RDEF file and native about dialog #38

Closed andimachovec closed 1 year ago

andimachovec commented 1 year ago

Added a menu bar with an "About" option which displays a standard about dialog. The version info is fetched from the rdef file. Having the app signature in the RDEF also make the app discoverable by QuickLaunch

andimachovec commented 1 year ago

fixes #30 At least the main part of it. Fetching the version number of the installed ffmpeg will have to wait a little bit, I'll have to make some changes to the class that launches the ffmpeg binary first

andimachovec commented 1 year ago

@scottmc : Could you take a look at this PR please, if you have the time ;-)

scottmc commented 1 year ago

I get a warning in ffpui-application.cpp:28:69: ISO C++ forbids converting a string constant to 'char*' Otherwise, seems to work as expected.

andimachovec commented 1 year ago

That compiler warning was always there from before I first touched the code. It´s fixed now ;-)