[PATCH] Fix for -vo quartz crash on Intel Mac
Hello, Here is a patch for libvo/vo_quartz.c. PlanarPixmapInfoYUV420 structure requires big-endian value, storing little-endian value causes crash on Intel Mac. This patch fixes that problem. -- Taihei Monma <tmkk@mac.com>
Hello, On Sat, Jun 30, 2007 at 03:47:28AM +0900, Taihei Monma wrote:
Here is a patch for libvo/vo_quartz.c. PlanarPixmapInfoYUV420 structure requires big-endian value, storing little-endian value causes crash on Intel Mac. This patch fixes that problem.
be2me_32 is the right function/macro to do the conversion in MPlayer IMO (#include "mpbswap.h"). Greetings, Reimar Döffinger
Hello, On 2007/06/30, at 5:00, Reimar Döffinger wrote:
be2me_32 is the right function/macro to do the conversion in MPlayer IMO (#include "mpbswap.h").
Thank you for the comment. Now here is a patch with recommended macros. -- Taihei Monma <tmkk@mac.com>
Hi, On 6/29/07, Taihei Monma <tmkk@mac.com> wrote:
Hello,
On 2007/06/30, at 5:00, Reimar Döffinger wrote:
be2me_32 is the right function/macro to do the conversion in MPlayer IMO (#include "mpbswap.h").
Thank you for the comment. Now here is a patch with recommended macros.
Ok, I will test it in a short while. Guillaume -- Subversion has been the most pointless project ever started -- Linus Torvalds
Hello, On Fri, Jun 29, 2007 at 09:35:34PM +0200, Guillaume POIRIER wrote:
On 6/29/07, Taihei Monma <tmkk@mac.com> wrote:
On 2007/06/30, at 5:00, Reimar Döffinger wrote:
be2me_32 is the right function/macro to do the conversion in MPlayer IMO (#include "mpbswap.h").
Thank you for the comment. Now here is a patch with recommended macros.
Ok, I will test it in a short while.
Please do so. It was tested on Intel Mac by someone on IRC and there it worked great. I was hoping to come up with a less ugly way, but didn't have a really great ideas. So I'll hope a few days for the official maintainer (or at least someone with a Mac) to commit, otherwise I will (and if it's not committed within one week at latest please remind me!). Greetings, Reimar Döffinger
On 29-Jun-07, at 5:24 PM, Reimar Döffinger wrote:
Hello, On Fri, Jun 29, 2007 at 09:35:34PM +0200, Guillaume POIRIER wrote:
On 6/29/07, Taihei Monma <tmkk@mac.com> wrote:
On 2007/06/30, at 5:00, Reimar Döffinger wrote:
be2me_32 is the right function/macro to do the conversion in MPlayer IMO (#include "mpbswap.h").
Thank you for the comment. Now here is a patch with recommended macros.
Ok, I will test it in a short while.
Please do so. It was tested on Intel Mac by someone on IRC and there it worked great. I was hoping to come up with a less ugly way, but didn't have a really great ideas. So I'll hope a few days for the official maintainer (or at least someone with a Mac) to commit, otherwise I will (and if it's not committed within one week at latest please remind me!).
Work great, patch submited. thanks ==== Nicolas Plourde <nicolas.plourde@gmail.com>
participants (4)
-
Guillaume POIRIER -
Nicolas Plourde -
Reimar Döffinger -
Taihei Monma