[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