Macintosh-HW/RASCSI
1
0
Fork 0
mirror of https://github.com/akuker/RASCSI.git synced 2024-12-23 06:30:04 +00:00
Code Issues Projects Releases Wiki Activity

Removed redundant message field

Browse Source
This commit is contained in:
Uwe Seimet 2021-12-17 23:19:08 +01:00
parent ff468aafa8
commit 49e14f7078
3 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/raspberrypi/rascsi_interface.proto
View File

@ -169,9 +169,8 @@ message PbOperationParameter {
string type = 3; string type = 3;
// There is no specific set of values if empty // There is no specific set of values if empty
repeated string values = 4; repeated string values = 4;
// Optional default value // Optional default value. If there is no default the value is mandatory.
string default_value = 5; string default_value = 5;
bool is_mandatory = 6;
} }
// The list of parameters supported by an operation // The list of parameters supported by an operation

1
src/raspberrypi/rascsi_response.cpp
View File

@ -457,7 +457,6 @@ PbOperationParameter *RascsiResponse::AddOperationParameter(PbOperationParameter
if (!default_value.empty()) { if (!default_value.empty()) {
parameter->set_default_value(default_value); parameter->set_default_value(default_value);
} }
parameter->set_is_mandatory(default_value.empty());
return parameter; return parameter;
} }

2
src/raspberrypi/rasctl_display.cpp
View File

@ -290,7 +290,7 @@ void RasctlDisplay::DisplayOperationInfo(const PbOperationInfo& operation_info)
for (const auto& parameter : operation.parameters()) { for (const auto& parameter : operation.parameters()) {
cout << " " << parameter.name() << ": " << parameter.type() cout << " " << parameter.name() << ": " << parameter.type()
<< (parameter.is_mandatory() ? ", mandatory" : ", optional"); << (parameter.default_value().empty() ? ", mandatory" : ", optional");
if (!parameter.description().empty()) { if (!parameter.description().empty()) {
cout << " (" << parameter.description().at("en") << ")"; cout << " (" << parameter.description().at("en") << ")";
} }