oaubert / python-vlc

Python vlc bindings
GNU Lesser General Public License v2.1
391 stars 111 forks source link

Regression in [OUT] parameters handling #287

Open oaubert opened 3 weeks ago

oaubert commented 3 weeks ago

The libvlc_log_get_context is improperly wrapped in the latest bindings version: parameters marked as [OUT] are not properly parsed as such, as indicated by the generate.py -dc output, as well as the generated code. The current version https://github.com/oaubert/python-vlc/blob/master/generated/3.0/vlc.py#L7305 states that the parameter signature is ( (1,), (1,), (1,), (2,), ), (3 in-parameters, 1 out parameter) while the previous version of the bindings (before the tree-sitter change) correctly gets ((1,), (2,), (2,), (2,),) : https://github.com/oaubert/python-vlc/blob/587d6263ba3176e158b6804f7457163ba1bed12c/generated/3.0/vlc.py#L5151

This is a regression, which also happens to other functions (e.g. libvlc_media_tracks_get, libvlc_media_slaves_get...) but sometimes is correct (e.g. libvlc_video_get_size, libvlc_video_get_cursor).