Streaming server MVP #8
Conversation
|
|
||
| set(CMAKE_CXX_STANDARD 17) | ||
|
|
||
| find_package(Boost 1.82.0 REQUIRED COMPONENTS filesystem) |
There was a problem hiding this comment.
используйте лучше std::filesystem если оно подходит по ваши нужды
| include_directories(boost_test ${Boost_INCLUDE_DIRS}) | ||
| find_package(Threads REQUIRED) | ||
|
|
||
| set(FFMPEG_LIBRARIES "/usr/lib/x86_64-linux-gnu/libavcodec.so;/usr/lib/x86_64-linux-gnu/libavformat.so;/usr/lib/x86_64-linux-gnu/libavutil.so;/usr/lib/x86_64-linux-gnu/libswscale.so") |
There was a problem hiding this comment.
Это можно делать через PkgConfig
find_package(PkgConfig REQUIRED)
pkg_check_modules(AVCODEC REQUIRED IMPORTED_TARGET libavcodec)
target_link_libraries(server PUBLIC
PkgConfig::AVCODEC)
|
|
||
| set(FFMPEG_LIBRARIES "/usr/lib/x86_64-linux-gnu/libavcodec.so;/usr/lib/x86_64-linux-gnu/libavformat.so;/usr/lib/x86_64-linux-gnu/libavutil.so;/usr/lib/x86_64-linux-gnu/libswscale.so") | ||
|
|
||
| SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -pthread") |
|
|
||
| # Set include directories | ||
| include_directories(${CMAKE_CURRENT_SOURCE_DIR}/include) | ||
| include_directories(${CMAKE_CURRENT_SOURCE_DIR}/lib/VideoStream/include) |
|
|
||
| # Add source files | ||
| set(SOURCES | ||
| ${CMAKE_CURRENT_SOURCE_DIR}/src/main.cpp |
There was a problem hiding this comment.
CMAKE_CURRENT_SOURCE_DIR можно не писать, относительные пути работают от CMAKE_CURRENT_SOURCE_DIR
|
|
||
| class AudioEncoder : IEncoder { | ||
| public: | ||
| AudioEncoder() = delete; |
There was a problem hiding this comment.
удаляется сам, когда вы объявляете любой конструктор
| class IEncoder { | ||
| public: | ||
| virtual ~IEncoder() = default; | ||
| virtual std::vector<uint8_t> encode(const uint8_t* data) = 0; |
|
|
||
| class VideoStream { | ||
| public: | ||
| const std::string DEFAULT_CODEC = "H264"; |
There was a problem hiding this comment.
зачем держать это в классе - оно потребляет память. Вынесите в константу в CPP файл
| } | ||
|
|
||
| void VideoStream::open(const std::string& filename) { | ||
| _video_encoder = std::make_shared<VideoEncoder>(1920, 1080, 30, 10000000); |
There was a problem hiding this comment.
лучше принимать IEncoder или IVideoEncoder как параметр, тогда это будет Dependency Injection, код будет меньше связан
| bool findStreamInfo(); | ||
|
|
||
| std::string file_path_; | ||
| AVFormatContext* format_context_ = nullptr; |
There was a problem hiding this comment.
лучше сделать под такие данные C++ RAII обертки, чтобы правильно из создавать и уничтожать
No description provided.