[MPlayer-dev-eng] [PATCH] Set default Blu-ray device

Brad Smith brad at comstyle.com
Mon May 6 07:43:49 CEST 2013


On Mon, May 06, 2013 at 07:40:03AM +0200, Reimar D??ffinger wrote:
> On 05.05.2013, at 23:14, Brad Smith <brad at comstyle.com> wrote:
> > On Sun, May 05, 2013 at 07:10:12PM +0200, Reimar D?ffinger wrote:
> >> On Sun, May 05, 2013 at 12:41:54AM -0400, Brad Smith wrote:
> >>> Submitting a local patch from our OpenBSD ports tree to set
> >>> a default Blu-ray device.
> >> 
> >> Since it's the same in all cases, why not just use DEFAULT_DVD_DEVICE?
> > 
> > You mean within stream_bluray.c?
> 
> Yes.

I had considered that. I wasn't sure if that would be Ok and just thought
I would submit the slightly simplified diff that I did and see where it
goes from there.

How about this then?


Index: stream/stream_bluray.c
===================================================================
--- stream/stream_bluray.c	(revision 36243)
+++ stream/stream_bluray.c	(working copy)
@@ -254,6 +254,8 @@
         device = p->device;
     else if (bluray_device)
         device = bluray_device;
+    else
+        device = DEFAULT_DVD_DEVICE;
 
     if (!device) {
         mp_msg(MSGT_OPEN, MSGL_ERR, MSGTR_BlurayNoDevice);

-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.



More information about the MPlayer-dev-eng mailing list