[haiku-development] Re: New Diff to Media player

Fredrik Modéen <fredrik@xxxxxxxxx> wrote:
> > 3) it makes more sense to pass a BString reference to this method,
> > as
> > you'll then save an unnecessary copy.
> and perhaps to _IsMedia and _IsPlayList aswell as I only are sending
> them
> to ReturnMIMEString.

If you can keep the info around easily for both calls that would indeed
be better.

> My Question are if making a BString are better than sending an
> existing
> entry_ref? I need the path to the file and the only way I found
> getting
> that was to make a BPath() and then call Path() to get a string. Can
> change
> ReturnMIMEString,  _IsMedia and _IsPlayList to take const char*
> insted of
> a BString.

Why would those methods need a path? If you want to check for the
suffix, the name of the file is enough, you don't need the path for
that (entry_ref::name, BEntry::GetName()).
Also, converting an entry to its full path (and back) isn't a cheap
operation.

Bye,
   Axel.

Other related posts: