Closed arcticwaters closed 8 years ago
This is a good approach and I’m keen to merge. Couple of things.
Unfortunately, I think the only thing this will help with is less code to convert to a Tizen app. I haven't taken a good look at Tizen yet, but I believe its a total rewrite of the APIs (including these higher-level APIs).
Note that some calls were not able to be converted. They had no higher level equivalent.
I believe the idea Samsung had behind these APIs was to have single interface that worked on all TVs instead of different calling methods per model. The bulk of it looks to catch C-series and below models, but there are a few for later models as well. Internally, they call almost the methods that we called before, but on a per-model basis. The documentation is worse than poor in most cases.
I'd rather have (2) in a separate pull. That will make it easier to identify this merge which will help d-series users from another which does not. I'd almost like to get this applied against the 1.x branch as well.
Closing to rebase pull on top of 2.2_dev.
I've moved a lot of the code which interacted directly with plugin objects to use higher-level APIs available in SmartHub. One useful byproduct that we can now support direct play of all audio streams in a container instead of just the first one. This initially had more steam for d-series users who were unable to transcode (and stuck with the first audio stream only), but is useful for other models as well.
Works well in the emulator. Unfortunately, I don't have a supported device to test on.
You want to test 3d as well, as the emulator doesn't support it.