Fix the easy comments.
This commit is contained in:
@@ -34,7 +34,7 @@ rational TimeBasedPanel::GetTime()
|
||||
return widget_->GetTime();
|
||||
}
|
||||
|
||||
rational TimeBasedPanel::timebase()
|
||||
const rational& TimeBasedPanel::timebase()
|
||||
{
|
||||
return widget_->timebase();
|
||||
}
|
||||
|
||||
@@ -41,7 +41,7 @@ public:
|
||||
|
||||
rational GetTime();
|
||||
|
||||
rational timebase();
|
||||
const rational& timebase();
|
||||
|
||||
ViewerOutput *GetConnectedViewer() const
|
||||
{
|
||||
|
||||
@@ -175,6 +175,7 @@ void MenuShared::AddItemsForTimeRulerMenu(Menu *m, const rational& timebase)
|
||||
|
||||
if (Timecode::TimebaseIsDropFrame(timebase)) {
|
||||
m->addAction(view_timecode_view_dropframe_item_);
|
||||
m->addAction(view_timecode_view_nondropframe_item_);
|
||||
} else {
|
||||
m->addAction(view_timecode_view_nondropframe_item_);
|
||||
}
|
||||
|
||||
@@ -288,7 +288,7 @@ void NodeParamViewWidgetBridge::WidgetCallback()
|
||||
{
|
||||
// Widget is a RationalSlider
|
||||
RationalSlider* slider = static_cast<RationalSlider*>(sender());
|
||||
ProcessSlider(slider, QVariant::fromValue(slider->GetValue()));;
|
||||
ProcessSlider(slider, QVariant::fromValue(slider->GetValue()));
|
||||
break;
|
||||
}
|
||||
case NodeValue::kVec2:
|
||||
@@ -528,8 +528,6 @@ rational NodeParamViewWidgetBridge::GetCurrentTimeAsNodeTime() const
|
||||
|
||||
void NodeParamViewWidgetBridge::SetTimebase(const rational& timebase)
|
||||
{
|
||||
timebase_ = timebase;
|
||||
|
||||
if (input_.GetDataType() == NodeValue::kRational) {
|
||||
static_cast<RationalSlider*>(widgets_.first())->SetTimebase(timebase);
|
||||
}
|
||||
|
||||
@@ -76,8 +76,6 @@ private:
|
||||
|
||||
rational time_;
|
||||
|
||||
rational timebase_;
|
||||
|
||||
NodeInputDragger dragger_;
|
||||
|
||||
NodeParamViewScrollBlocker scroll_filter_;
|
||||
|
||||
@@ -303,10 +303,7 @@ void MainMenu::ViewMenuAboutToShow()
|
||||
// Parent is QMainWindow
|
||||
view_full_screen_item_->setChecked(parentWidget()->isFullScreen());
|
||||
|
||||
// Ensure checked timecode display mode is correct
|
||||
MenuShared::instance()->AboutToShowTimeRulerActions();
|
||||
|
||||
|
||||
// Make sure we're displaying the correct options for the timebase
|
||||
TimeBasedPanel* p = PanelManager::instance()->MostRecentlyFocused<TimeBasedPanel>();
|
||||
if (p) {
|
||||
if (p->timebase().denominator() != 0) {
|
||||
@@ -314,6 +311,9 @@ void MainMenu::ViewMenuAboutToShow()
|
||||
MenuShared::instance()->AddItemsForTimeRulerMenu(view_menu_, p->timebase());
|
||||
}
|
||||
}
|
||||
|
||||
// Ensure checked timecode display mode is correct
|
||||
MenuShared::instance()->AboutToShowTimeRulerActions();
|
||||
}
|
||||
|
||||
void MainMenu::ToolsMenuAboutToShow()
|
||||
|
||||
Reference in New Issue
Block a user