[vlc-devel] BeOS interface CVS Commit
Stephan Assmus
stippi at yellowbites.com
Sat Feb 1 13:15:16 CET 2003
Hello all,
sorry for commiting without useful message! Here is what my changes
were:
modules/gui/beos/Interface.cpp
- changed heap allocation of a BScreen object to a stack allocation
reason: no good reason, just a habbit, and I fixed many leaks in
the past when the allocated object was not deleted
modules/gui/beos/InterfaceWindow.h/.cpp
- fixed a couple of memory leaks
-> please consult BeBook if parameters passed to functions remain
the callers responsibility to delete!
- fixed a bunch of bugs
- removed usage of second file panel by sharing one file panel for
multiple tasks
reason: user would have to navigate into a certain folder twice,
to select a divx movie and the corresponding subtile file
plus it's a waste of resources
- changed a couple strings in the interface (eg "Show"->"Window"),
if similar strings are freaquently used in other programs
- reverted saving of UI settings to BeOS way
reason: I had problems with the vlc way of saving them, plus the
BeOS way gives some advantages, like forward compatibility and
better error checking possibilities/default values
however, I did not remove the other code, maybe I make both
ways work right and selectable.
- more UI settings are saved
- code clean up, and migration of some useful code from 0.4.6
modules/gui/beos/ListViews.h/.cpp
- more efficient/robust drag'n'drop indication code
- modification of vlc playlist via drag'n'drop
- improved class design in some parts
- moved code arround from PlaylistWindow.cpp
modules/gui/beos/MediaControlView.cpp
- fixed some locking bugs (crashes):
wrong:
LockLooper();
...
UnlockLooper();
right:
if (LockLooper())
{
...
UnlockLooper();
}
these Be routines have return codes for a reason!
modules/gui/beos/MessagesWindow.cpp
- see above
modules/gui/beos/PlayListWindow.h/.cpp
- implemented some of the previously disabled menu items in edit menu
- Select All/None
- Remove (All)
modules/gui/beos/VideoWindow.h/VideoOutput.cpp
- reimplemented saving of video settings with 0.4.6 code
-> why got it removed without replacement?!?
-> if it has to be the vlcrc file, just implement the saving
in that way, don't remove useful code!
modules/gui/beos/VlcWrapper.h/.cpp
- spotted some bugs
- added methods for modifying the playlist
(only PlaylistView class uses them)
- changed OpenFiles() method to allow for insertion of items
at any position (as used when files are dropped into playlist)
--
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html
If you are in trouble, please contact <postmaster at videolan.org>
More information about the vlc-devel
mailing list