[vlc-devel] [PATCH 0/3] upnp: refactor out UpnpInstanceWrapper

Shaleen Jain shaleen at jain.sh
Mon Jun 25 15:49:09 CEST 2018


This refactor is required to be able to have multiple
modules utilizing the libupnp library within VLC
considering the existing constraints of having only
one instance of libupnp per process.

Motivation:
I plan to add another module (a stream_out module)
leveraging upnp functionality without having to 
stuff it in an already crammed upnp.cpp

Shaleen Jain (3):
  upnp: add and use a callback listener interface
  upnp: move UpnpInstanceWrapper to upnp-wrapper
  upnp: rename module callbacks to be more verbose

 modules/services_discovery/Makefile.am      |   4 +-
 modules/services_discovery/upnp-wrapper.cpp | 156 +++++++
 modules/services_discovery/upnp-wrapper.hpp | 349 ++++++++++++++
 modules/services_discovery/upnp.cpp         | 487 ++------------------
 modules/services_discovery/upnp.hpp         |  62 +--
 5 files changed, 555 insertions(+), 503 deletions(-)
 create mode 100644 modules/services_discovery/upnp-wrapper.cpp
 create mode 100644 modules/services_discovery/upnp-wrapper.hpp

-- 
2.18.0


More information about the vlc-devel mailing list