Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

decode: added ability to reuse device for multiple files #120

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

t-boiko
Copy link
Contributor

@t-boiko t-boiko commented Dec 3, 2024

  • added an ability to process several files in order by the same process in a linear order;
  • added an ability to launch multiple decode processes and to handle them using a unix domain sockets.

@t-boiko t-boiko force-pushed the decoder-multisession-mode branch 2 times, most recently from 1f0629c to 37888c5 Compare December 3, 2024 11:10
Copy link
Contributor

@zlatinski zlatinski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I need to spend more time reviewing the change. These are just a few comments on code clean-up and code style.

if (result != VK_SUCCESS) {
return -1;
}
// result = VulkanVideoProcessor::Create(&vkDevCtxt, vulkanVideoProcessor); // moved closer to a decodeLoop
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please remove all this commented-out code?

// }
// }

// //Call WSAPoll for writeability of accepted
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here too - remove this block of commented code, please.

data_sent = 1;
}
if (ret)
{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use a code formatter to fix all these inconsistencies of code style.

@t-boiko t-boiko force-pushed the decoder-multisession-mode branch 3 times, most recently from b406a0e to a5da5b6 Compare December 4, 2024 16:51
@t-boiko t-boiko force-pushed the decoder-multisession-mode branch from a5da5b6 to 5189559 Compare December 4, 2024 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants