Skip to content

Commit

Permalink
Fix for review comment
Browse files Browse the repository at this point in the history
  • Loading branch information
ZitaLiao committed Aug 31, 2023
1 parent 3ab9d06 commit 43c9ea3
Show file tree
Hide file tree
Showing 8 changed files with 13 additions and 11 deletions.
6 changes: 4 additions & 2 deletions include/rtc/h264rtppacketizer.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,13 @@ class RTC_CPP_EXPORT H264RtpPacketizer final : public RtpPacketizer,
shared_ptr<NalUnits> splitMessage(binary_ptr message);
const uint16_t maximumFragmentSize;

using Separator=NalUnit::Separator;

public:
/// Default clock rate for H264 in RTP
inline static const uint32_t defaultClockRate = 90 * 1000;

H264RtpPacketizer(NalUnit::Separator separator,
H264RtpPacketizer(Separator separator,
shared_ptr<RtpPacketizationConfig> rtpConfig,
uint16_t maximumFragmentSize = NalUnits::defaultMaximumFragmentSize);

Expand All @@ -43,7 +45,7 @@ class RTC_CPP_EXPORT H264RtpPacketizer final : public RtpPacketizer,
message_ptr control) override;

private:
const NalUnit::Separator separator;
const Separator separator;
};

} // namespace rtc
Expand Down
2 changes: 1 addition & 1 deletion include/rtc/h265nalunit.hpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
* Copyright (c) 2020 Filip Klembara (in2core)
* Copyright (c) 2023 Zita Liao (Dolby)
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
Expand Down
2 changes: 1 addition & 1 deletion include/rtc/h265packetizationhandler.hpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
* Copyright (c) 2020 Filip Klembara (in2core)
* Copyright (c) 2023 Zita Liao (Dolby)
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
Expand Down
2 changes: 1 addition & 1 deletion include/rtc/h265rtppacketizer.hpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
* Copyright (c) 2020 Filip Klembara (in2core)
* Copyright (c) 2023 Zita Liao (Dolby)
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
Expand Down
6 changes: 3 additions & 3 deletions src/h264rtppacketizer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ namespace rtc {

shared_ptr<NalUnits> H264RtpPacketizer::splitMessage(binary_ptr message) {
auto nalus = std::make_shared<NalUnits>();
if (separator == NalUnit::Separator::Length) {
if (separator == Separator::Length) {
size_t index = 0;
while (index < message->size()) {
assert(index + 4 < message->size());
Expand Down Expand Up @@ -83,9 +83,9 @@ shared_ptr<NalUnits> H264RtpPacketizer::splitMessage(binary_ptr message) {
H264RtpPacketizer::H264RtpPacketizer(shared_ptr<RtpPacketizationConfig> rtpConfig,
uint16_t maximumFragmentSize)
: RtpPacketizer(rtpConfig), MediaHandlerRootElement(), maximumFragmentSize(maximumFragmentSize),
separator(NalUnit::Separator::Length) {}
separator(Separator::Length) {}

H264RtpPacketizer::H264RtpPacketizer(NalUnit::Separator separator,
H264RtpPacketizer::H264RtpPacketizer(Separator separator,
shared_ptr<RtpPacketizationConfig> rtpConfig,
uint16_t maximumFragmentSize)
: RtpPacketizer(rtpConfig), MediaHandlerRootElement(), maximumFragmentSize(maximumFragmentSize),
Expand Down
2 changes: 1 addition & 1 deletion src/h265nalunit.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
* Copyright (c) 2020 Filip Klembara (in2core)
* Copyright (c) 2023 Zita Liao (Dolby)
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
Expand Down
2 changes: 1 addition & 1 deletion src/h265packetizationhandler.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
* Copyright (c) 2020 Filip Klembara (in2core)
* Copyright (c) 2023 Zita Liao (Dolby)
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
Expand Down
2 changes: 1 addition & 1 deletion src/h265rtppacketizer.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
* Copyright (c) 2020 Filip Klembara (in2core)
* Copyright (c) 2023 Zita Liao (Dolby)
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
Expand Down

0 comments on commit 43c9ea3

Please sign in to comment.