December 2008 - MPlayer-dev-eng - lists.mplayerhq.hu

[PATCH] add support of compression algorithm 3 in mkv demuxer
by Alexander Beregalov 02 Jan '12

02 Jan '12

Specs from http://www.matroska.org/technical/specs/ : ContentCompAlgo The compression algorithm used. Algorithms that have been specified so far are: 0 - zlib, 1 - bzlib, 2 - lzo1x, 3 - Header Stripping. ContentCompSettings Settings that might be needed by the decompressor. For Header Stripping (ContentCompAlgo=3), the bytes that were removed from the beggining of each frames of the track. I have such file, it works for me. | + A track | + Track number: 2 | + Track UID: 2 | + Track type: audio | + Lacing flag: 1 | + Default flag: 1 | + Codec ID: A_AC3 | + Content encodings | + Content encoding | + Content compression | + Algorithm: 3 (header removal) | + Settings: length 2, data: 0x0b 0x77 See an attachment. Please comment.

7 21

rtp support in mplayer
by chas williams - CONTRACTOR 30 Apr '10

30 Apr '10

while trying to play from rtp/mpeg-es streams i found i needed the following patch: --- stream/rtp.c.000 2008-05-13 17:37:23.000000000 -0400 +++ stream/rtp.c 2008-05-13 18:02:02.000000000 -0400 @@ -246,6 +246,9 @@ headerSize = 12 + 4*rh->b.cc; /* in bytes */ + if (rh->b.pt == 32) /* 32 = MPV from RFC 1890 */ + headerSize += 4; /* skip MPEG video-specific header RFC 2250 */ + *lengthData = lengthPacket - headerSize; *data = (char*) buf + headerSize; to skip over the 4-byte mpeg header present in the file. however, after looking at mplayer a bit harder, i see that there are two rtp decoders/demuxers. the one supplied in libavformat (apparently derived from ffmpeg) seems like it has more smarts. which one should i be using?

4 13

Another sample from norway
by Håkon Alstadheim 27 Aug '09

27 Aug '09

This is just to motivate anybody having a spare 5 minutes into working receiving the norwegian dvb-t broadcasts ... (should also interest people from NZ) :-) I've uploaded another sample to incoming, filename hakon.alstadheim.priv.no.nrk3.ts. Attaching the standard output and error when playing below. As you can see I have patched my mplayer to force 25 fps framerate. Seems that patch no longer works as intended, but the straight (non-patched) mplayer gives similarily discouraging results when viewing. Happy hacking everybody :-) . If you want more debugging output, just tell me what and I'll try to comply. -------------------- MPlayer dev-SVN-r26171-4.1.2 (C) 2000-2008 MPlayer Team CPU: Intel(R) Pentium(R) 4 CPU 2.00GHz (Family: 15, Model: 2, Stepping: 4) CPUflags: MMX: 1 MMX2: 1 3DNow: 0 3DNow2: 0 SSE: 1 SSE2: 1 Compiled for x86 CPU with extensions: MMX MMX2 SSE SSE2 Playing hakon.alstadheim.priv.no.nrk3.ts. TS file format detected. VIDEO H264(pid=521) AUDIO AAC(pid=676) NO SUBS (yet)! PROGRAM N. 103 FPS seems to be: 25.000000 ========================================================================== Opening video decoder: [ffmpeg] FFmpeg's libavcodec codec family Selected video codec: [ffh264] vfm: ffmpeg (FFmpeg H.264) ========================================================================== ========================================================================== Opening audio decoder: [faad] AAC (MPEG2/4 Advanced Audio Coding) versionA not supported FAAD: compressed input bitrate missing, assuming 128kbit/s! AUDIO: 44100 Hz, 2 ch, s16le, 128.0 kbit/9.07% (ratio: 16000->176400) Selected audio codec: [faad] afm: faad (FAAD AAC (MPEG-2/MPEG-4 Audio) decoder) ========================================================================== AO: [oss] 44100Hz 2ch s16le (2 bytes per sample) Starting playback... FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: Failed to decode frame: Invalid number of channels [h264 @ 0x88d3070]left block unavailable for requested intra mode at 0 4 [h264 @ 0x88d3070]error while decoding MB 0 4, bytestream (24666) FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Quantised value out of range, trying to resync! FAAD: error: Quantised value out of range, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: Failed to decode frame: Gain control not yet implemented VDec: vo config request - 720 x 576 (preferred colorspace: Planar YV12) VDec: using Planar YV12 as output csp (no 0) Movie-Aspect is 1.82:1 - prescaling to correct movie aspect. VO: [xv] 720x576 => 1048x576 Planar YV12 [ASPECT] Warning: No suitable new res found! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Unexpected fill element with SBR data, trying to resync! FAAD: Failed to decode frame: Unexpected fill element with SBR data [h264 @ 0x88d3070]B picture before any references, skipping [h264 @ 0x88d3070]decode_slice_header error [h264 @ 0x88d3070]no frame! Error while decoding frame! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Scalefactor out of range, trying to resync! FAAD: error: Pulse coding not allowed in short blocks, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: Failed to decode frame: Gain control not yet implemented [h264 @ 0x88d3070]B picture before any references, skipping [h264 @ 0x88d3070]decode_slice_header error [h264 @ 0x88d3070]no frame! Error while decoding frame! FAAD: error: Invalid number of channels, trying to resync! ??,?% 0 0 FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: Failed to decode frame: Maximum number of scalefactor bands exceeded [h264 @ 0x88d3070]B picture before any references, skipping [h264 @ 0x88d3070]decode_slice_header error [h264 @ 0x88d3070]no frame! Error while decoding frame! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Gain control not yet implemented, trying to resync! FAAD: error: Channel coupling not yet implemented, trying to resync! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: error: Invalid number of channels, trying to resync! FAAD: Failed to decode frame: Invalid number of channels [h264 @ 0x88d3070]B picture before any references, skipping [h264 @ 0x88d3070]decode_slice_header error [h264 @ 0x88d3070]no frame! Error while decoding frame! FAAD: error: Maximum number of scalefactor bands exceeded, trying to resync! FAAD: error: Quantised value out of range, trying to resync! MPlayer interrupted by signal 11 in module: decode_audio - MPlayer crashed by bad usage of CPU/FPU/RAM. Recompile MPlayer with --enable-debug and make a 'gdb' backtrace and disassembly. Details in DOCS/HTML/en/bugreports_what.html#bugreports_crash. - MPlayer crashed. This shouldn't happen. It can be a bug in the MPlayer code _or_ in your drivers _or_ in your gcc version. If you think it's MPlayer's fault, please read DOCS/HTML/en/bugreports.html and follow the instructions there. We can't and won't help unless you provide this information when reporting a possible bug.

3 5

Direct 3D Windows VO
by Olivier Chalouhi 17 Feb '09

17 Feb '09

Are there any plans for developing a direct 3d (vs directx) vo for windows. The current directx vo on Vista forces the compatibility mode and disables aero, which results in a degraded user experience. Moreover the current directx implementation will only work with the primary display. If anyone would have the capabilities of doing it (VLC has one, so it may be possible to base the mplayer's one on the implementation from VLC), please contact me back, as my company would be willing to pay for this development. Best, Olivier Chalouhi.

6 23

[PATCH] - No Reset of D3D device on resize
by Jim Hauxwell 27 Jan '09

27 Jan '09

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hi, I've knocked up this small patch so that the buffers don't have to be torn down (the display surfaces) and the device doesn't need to be reset. I've tested in SMplayer and it seems to work much better, smooth resize without flickering. Its just a first stab as it only took 1/2 hour. If anyone wants to test, here it is. Jim -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (MingW32) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iEYEARECAAYFAklBohcACgkQhrNWoHjgI1CVxgCgsymbkWVJxTFfoF1TENGv9CJ4 zkwAn1LwQwVczYBHPA4oP6Tb9JOVSDyb =bmrA -----END PGP SIGNATURE----- Index: libvo/vo_direct3d.c =================================================================== --- libvo/vo_direct3d.c (revision 28127) +++ libvo/vo_direct3d.c (working copy) @@ -202,14 +202,6 @@ mp_msg(MSGT_VO, MSGL_V, "<vo_direct3d>destroy_d3d_surfaces called\r\n"); /* Let's destroy the old (if any) D3D Surfaces */ - if (priv->locked_rect.pBits) - IDirect3DSurface9_UnlockRect(priv->d3d_surface); - priv->locked_rect.pBits = NULL; - - if (priv->d3d_surface) - IDirect3DSurface9_Release(priv->d3d_surface); - priv->d3d_surface = NULL; - /* kill the OSD texture and its shadow copy */ if (priv->d3d_texture_osd) IDirect3DTexture9_Release(priv->d3d_texture_osd); @@ -218,10 +210,6 @@ if (priv->d3d_texture_system) IDirect3DTexture9_Release(priv->d3d_texture_system); priv->d3d_texture_system = NULL; - - if (priv->d3d_backbuf) - IDirect3DSurface9_Release(priv->d3d_backbuf); - priv->d3d_backbuf = NULL; } /** @brief Create D3D Offscreen and Backbuffer surfaces. @@ -233,21 +221,6 @@ int tex_width = osd_width, tex_height = osd_height; mp_msg(MSGT_VO, MSGL_V, "<vo_direct3d><INFO>create_d3d_surfaces called.\n"); - if (FAILED(IDirect3DDevice9_CreateOffscreenPlainSurface( - priv->d3d_device, priv->src_width, priv->src_height, - priv->movie_src_fmt, D3DPOOL_DEFAULT, &priv->d3d_surface, NULL))) { - mp_msg(MSGT_VO, MSGL_ERR, - "<vo_direct3d><INFO>IDirect3D9_CreateOffscreenPlainSurface Failed.\n"); - return 0; - } - - if (FAILED(IDirect3DDevice9_GetBackBuffer(priv->d3d_device, 0, 0, - D3DBACKBUFFER_TYPE_MONO, - &priv->d3d_backbuf))) { - mp_msg(MSGT_VO, MSGL_ERR, "<vo_direct3d>Back Buffer address get failed\n"); - return 0; - } - /* calculate the best size for the OSD depending on the factors from the device */ if (priv->device_caps_power2_only) { tex_width = 1; @@ -332,8 +305,8 @@ present_params->SwapEffect = D3DSWAPEFFECT_COPY; present_params->Flags = D3DPRESENTFLAG_VIDEO; present_params->hDeviceWindow = vo_w32_window; /* w32_common var */ - present_params->BackBufferWidth = 0; /* Fill up window Width */ - present_params->BackBufferHeight = 0; /* Fill up window Height */ + present_params->BackBufferWidth = priv->max_texture_width; /* Fill up window Width */ + present_params->BackBufferHeight = priv->max_texture_height; /* Fill up window Height */ present_params->MultiSampleType = D3DMULTISAMPLE_NONE; /* D3DPRESENT_INTERVAL_ONE = vsync */ present_params->PresentationInterval = D3DPRESENT_INTERVAL_ONE; @@ -350,6 +323,7 @@ { D3DPRESENT_PARAMETERS present_params; D3DDISPLAYMODE disp_mode; + D3DVIEWPORT9 vp = {0, 0, vo_dwidth, vo_dheight, 0, 1}; mp_msg(MSGT_VO, MSGL_V, "<vo_direct3d><INFO>configure_d3d called\n"); @@ -382,11 +356,27 @@ if (!create_d3d_surfaces()) return 0; - mp_msg(MSGT_VO, MSGL_V, - "New BackBuffer: Width: %d, Height:%d. VO Dest Width:%d, Height: %d\n", - present_params.BackBufferWidth, present_params.BackBufferHeight, - vo_dwidth, vo_dheight); + if (FAILED(IDirect3DDevice9_CreateOffscreenPlainSurface( + priv->d3d_device, priv->src_width, priv->src_height, + priv->movie_src_fmt, D3DPOOL_DEFAULT, &priv->d3d_surface, NULL))) { + mp_msg(MSGT_VO, MSGL_ERR, + "<vo_direct3d><INFO>IDirect3D9_CreateOffscreenPlainSurface Failed.\n"); + return 0; + } + if (FAILED(IDirect3DDevice9_GetBackBuffer(priv->d3d_device, 0, 0, + D3DBACKBUFFER_TYPE_MONO, + &priv->d3d_backbuf))) { + mp_msg(MSGT_VO, MSGL_ERR, "<vo_direct3d>Back Buffer address get failed\n"); + return 0; + } + + if (FAILED(IDirect3DDevice9_SetViewport(priv->d3d_device, + &vp))) { + mp_msg(MSGT_VO, MSGL_ERR, "<vo_direct3d>Unable to set the viewport\n"); + return VO_ERROR; + } + calc_fs_rect(); return 1; @@ -400,6 +390,14 @@ { mp_msg(MSGT_VO, MSGL_V, "<vo_direct3d><INFO>reconfigure_d3d called.\n"); + if (priv->locked_rect.pBits) + IDirect3DSurface9_UnlockRect(priv->d3d_surface); + priv->locked_rect.pBits = NULL; + + if (priv->d3d_surface) + IDirect3DSurface9_Release(priv->d3d_surface); + priv->d3d_surface = NULL; + /* Destroy the Offscreen and Backbuffer surfaces */ destroy_d3d_surfaces(); @@ -431,30 +429,21 @@ */ static int resize_d3d(void) { - D3DPRESENT_PARAMETERS present_params; + D3DVIEWPORT9 vp = {0, 0, vo_dwidth, vo_dheight, 0, 1}; mp_msg(MSGT_VO, MSGL_V, "<vo_direct3d><INFO>resize_d3d called.\n"); + if (FAILED(IDirect3DDevice9_SetViewport(priv->d3d_device, + &vp))) { + mp_msg(MSGT_VO, MSGL_ERR, "<vo_direct3d>Unable to set the viewport\n"); + return VO_ERROR; + } + destroy_d3d_surfaces(); - /* Reset the D3D Device with all parameters the same except the new - * width/height. - */ - fill_d3d_presentparams(&present_params); - if (FAILED(IDirect3DDevice9_Reset(priv->d3d_device, &present_params))) { - mp_msg(MSGT_VO, MSGL_ERR, - "<vo_direct3d><INFO>Could not reset the D3D device\n"); - return 0; - } - if (!create_d3d_surfaces()) return 0; - mp_msg(MSGT_VO, MSGL_V, - "New BackBuffer: Width: %d, Height:%d. VO Dest Width:%d, Height: %d\n", - present_params.BackBufferWidth, present_params.BackBufferHeight, - vo_dwidth, vo_dheight); - calc_fs_rect(); #ifdef CONFIG_FREETYPE @@ -473,6 +462,18 @@ { mp_msg(MSGT_VO, MSGL_V, "<vo_direct3d>uninit_d3d called\r\n"); + if (priv->locked_rect.pBits) + IDirect3DSurface9_UnlockRect(priv->d3d_surface); + priv->locked_rect.pBits = NULL; + + if (priv->d3d_surface) + IDirect3DSurface9_Release(priv->d3d_surface); + priv->d3d_surface = NULL; + + if (priv->d3d_backbuf) + IDirect3DSurface9_Release(priv->d3d_backbuf); + priv->d3d_backbuf = NULL; + destroy_d3d_surfaces(); /* Destroy the D3D Device */ @@ -787,7 +788,8 @@ */ static void flip_page(void) { - if (FAILED(IDirect3DDevice9_Present(priv->d3d_device, 0, 0, 0, 0))) { + RECT rect = {0, 0, vo_dwidth, vo_dheight}; + if (FAILED(IDirect3DDevice9_Present(priv->d3d_device, &rect, 0, 0, 0))) { mp_msg(MSGT_VO, MSGL_V, "<vo_direct3d>Video adapter became uncooperative.\n"); mp_msg(MSGT_VO, MSGL_ERR, "<vo_direct3d>Trying to reinitialize it...\n"); @@ -795,7 +797,7 @@ mp_msg(MSGT_VO, MSGL_V, "<vo_direct3d>Reinitialization Failed.\n"); return; } - if (FAILED(IDirect3DDevice9_Present(priv->d3d_device, 0, 0, 0, 0))) { + if (FAILED(IDirect3DDevice9_Present(priv->d3d_device, &rect, 0, 0, 0))) { mp_msg(MSGT_VO, MSGL_V, "<vo_direct3d>Reinitialization Failed.\n"); return; }

4 26

libass behavior when PlayResX & PlayResY are missing in ass subs
by Robert Ramiega 14 Jan '09

14 Jan '09

Hi! Lately i've been playing with ass subtitles and i've noticed that when subtitle file doesn't specify PlayResX *and* PlayResY libass hardcodes them to 384x288. What's the reasoning with this? I think this is a bit against SSA/ASS specs I've found on the net: PlayResY: This is the height of the screen used by the script's author(s) when playing the script. SSA v4 will automatically select the nearest enabled setting, if you are using Directdraw playback. PlayResX: This is the width of the screen used by the script's author(s) when playing the script. SSA will automatically select the nearest enabled, setting if you are using Directdraw playback. Locally I've modified ass_render.c so that if both values are missing in the sub file they are set to orig_width, orig_height (this seems to be at least a bit consistent with situation when either PlayResX or PlayResY is given -- see ass_lazy_track_init() around line 210 of ass_render.c ). -- Just a friendly Jedi Knight | I find your lack of faith Robert Ramiega jedi(a)plukwa.net | disturbing http://nlj.plukwa.net/

5 10

[PATCH] Fix in Documentation libmpcodecs.txt
by Vineeth N 11 Jan '09

11 Jan '09

Hello, If have a fix related to filename in the documentation, libmpcodecs.txt Regards, Vineeth

2 1

[PATCH] internal dvdnav
by Reimar Döffinger 09 Jan '09

09 Jan '09

Hello, with some more small changes to libdvdread and libdvdnav which are currently pending, and these two new svn externals: libdvdread svn://svn.mplayerhq.hu/dvdnav/trunk/libdvdread/src libdvdnav svn://svn.mplayerhq.hu/dvdnav/trunk/libdvdnav/src MPlayer can be built with (a working) embedded libdvdnav, and as far as I can tell it will use the embedded libdvdcss, too. A patch for that is attached. Greetings, Reimar Döffinger

4 13

N fps and N Hz screen refresh rate (the vblank problem)
by Pallai Roland 08 Jan '09

08 Jan '09

Hi, Preface: it's not about tearing, my sync-to-vblank on XV is OK. :) A long-time headache for me that movies played from my HTPC (on LCD TV) suffers some jittery. Now I've got some time for this problem, thanks for the boring Xmas. I came to realize that my problem is the refresh rate, that was 60Hz by default. It's good only for porn. :) I believed that the problem is solved when I set the refresh rate identical to video framerate, but in every 1-2 minutes the video jittered very badly for 10-15 seconds. The result of this experiment is this mail. The first step to play videos without jittering is to set identical (or multiple) refresh rate to the movie's fps. The multiple refresh rate is bad for the "pixel plus", "motion flow" engines (every manufacturer calls it different) of modern TVs. My Samsung M87 LCD TV accepts refresh rate 24/25/50/60hz on HDMI, optimal to drive it at 24/25hz when I watch movies, as a Blueray player box does. The problem if the refresh rate identical to video framerate and the (m)player ignores the vblank intervals and puts the frames on the brink of vblanks, then the result is a badly jittering video for some seconds, until the timing of video frames drifts away. It depends on luck, that a frame will be overwritten within a vblank or the player misses a vblank to put a video frame. Let me draw it, my english is sucks to express myself in text: :) first line: vblanks second line: update_video |1111111111|22222222222|3333333333|4444444444|5555555555|6666666666| | ^ |^ |^ ^| ^| |^ jitter! jitter! my mplayer patch does the following things: 1., reads the vblank informations from a regular file in adjust_sync_and_print_status(). file written by my drmsync.c that sucks the vblank timing from kernel's DRM. 2., notifies the user on OSD if a vblank has double update or a vblank missed. 3., tries to keep update_video in the first half of vblanks (the cpu latency -late frames- tolerated better in this way) My patch avoids from brink of vblanks when fps and hz identical, so no jitter in the situation above: first line: vblanks second line: update_video |1111111111|22222222222|3333333333|4444444444|5555555555|6666666666| | ^ |^ | ^ | ^ | ^ | ^ speed up video! (a little bit) The skip of a vblank is unavoidable sometimes (we've to keep audio in sync, unfortunately), but it's a little impact and rare. This patch is just a demonstration. It worksforme, my problem has gone, but it's an ugly hack. Is there another solution for this problem? Is a workaround? Is a plan to fix this issue in the mainline? ps.: Xine does ugly jittering in the whole movie when I tried play N fps at N hz. Maybe there's serious video frame timing issue. ps.: MythTV cares about vblank, http://www.cuymedia.net/mythtv-trunk/vsync_8cpp-source.html -- dap

5 7

[PATCH] Fontconfig -font support for font_face selection
by Adrian Stutz 06 Jan '09

06 Jan '09

Hello, When using Fontconfig and passing a pattern using -font, it's possible to select a font as well as a style. If the font styles are in one file however, the style selection gets lost because MPlayer only passes the filename to FreeType and selects the first face in the file. This patch gets the face index from Fontconfig and passes it along to Freetype to ensure the correct style is chosen. To reproduce, try to select a suitcase font style, like Palatino: mplayer -fontconfig -font "Palatino" mplayer -fontconfig -font "Palatino:Bold" mplayer -fontconfig -font "Palatino:style=Italic" all three display Palatino Bold Italic instead of Regular, Bold and Italic. With the patch, the correct style is selected. Greetings Adrian

2 1