[MPlayer-dev-eng] [PATCH] Windows DLL support for OS X/Intel (cleaned version)

Dominik 'Rathann' Mierzejewski dominik at rangers.eu.org
Tue Nov 21 03:58:33 CET 2006


On Tuesday, 21 November 2006 at 01:32, Nicolas Plourde wrote:
> On 11/20/06, Dominik 'Rathann' Mierzejewski <dominik at rangers.eu.org> wrote:
> >
> >
> >That's to be expected. If loader uses symbols from osdep then osdep needs
> >to be after loader in the link command.
> >
> >R.
> >
> >
> corrected version
[...]
> Index: loader/ldt_keeper.c
> ===================================================================
> --- loader/ldt_keeper.c	(revision 21117)
> +++ loader/ldt_keeper.c	(working copy)
> @@ -28,6 +28,7 @@
>  #include <sys/types.h>
>  #include <stdio.h>
>  #include <unistd.h>
> +#include "osdep/mmap_anon.h"
>  #ifdef __linux__
>  #include <asm/unistd.h>
>  #include <asm/ldt.h>
> @@ -200,8 +201,8 @@
>  	return NULL;
>      }
>      fs_seg=
> -    ldt_fs->fs_seg = mmap(NULL, getpagesize(), PROT_READ | PROT_WRITE, MAP_PRIVATE,
> -			  ldt_fs->fd, 0);
> +    ldt_fs->fs_seg = mmap_anon(NULL, getpagesize(), PROT_READ | PROT_WRITE, MAP_PRIVATE, 0, 
> +                &ldt_fs->fd);

Was there any reason to change the parameter order in mmap_anon?

Regards,
R.

-- 
MPlayer developer and RPMs maintainer: http://rpm.greysector.net/mplayer/
There should be a science of discontent. People need hard times and
oppression to develop psychic muscles.
	-- from "Collected Sayings of Muad'Dib" by the Princess Irulan



More information about the MPlayer-dev-eng mailing list