oak-team/oak-editor
3
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

suppress some benign warnings

Browse Source
This commit is contained in:
itsmattkc
2021-02-21 10:23:06 +11:00
parent e0443e017c
commit 7fee0e6b04
3 changed files with 13 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/node/value.cpp
+2 -1
View File
@@ -68,7 +68,8 @@ QString NodeValue::ValueToString(Type data_type, const QVariant &value, bool val
} else if (data_type == kRational) { } else if (data_type == kRational) {
return value.value<rational>().toString(); return value.value<rational>().toString();
} else if (data_type == kTexture } else if (data_type == kTexture
|| data_type == kSamples) { || data_type == kSamples
|| data_type == kNone) {
// These data types need no XML representation // These data types need no XML representation
return QString(); return QString();
} else if (data_type == kInt) { } else if (data_type == kInt) {
app/widget/curvewidget/curvewidget.cpp
+3 -1
View File
@@ -220,7 +220,9 @@ void CurveWidget::UpdateBridgeTime(const int64_t &timestamp)
void CurveWidget::ConnectNode(Node *node, bool connect) void CurveWidget::ConnectNode(Node *node, bool connect)
{ {
foreach (const QString& input, node->inputs()) { foreach (const QString& input, node->inputs()) {
ConnectInput(node, input, connect); if (node->IsInputKeyframable(input)) {
ConnectInput(node, input, connect);
}
} }
// Connect add/remove signals // Connect add/remove signals
app/widget/nodetableview/nodetableview.cpp
+8
View File
@@ -140,6 +140,14 @@ void NodeTableView::SetTime(const rational &time)
sub_item->setText(1, source_name); sub_item->setText(1, source_name);
switch (value.type()) { switch (value.type()) {
case NodeValue::kVideoParams:
case NodeValue::kAudioParams:
case NodeValue::kFootageJob:
case NodeValue::kShaderJob:
case NodeValue::kSampleJob:
case NodeValue::kGenerateJob:
// These types have no string representation
break;
case NodeValue::kTexture: case NodeValue::kTexture:
{ {
// NodeTableTraverser puts video params in here // NodeTableTraverser puts video params in here