Skip to content

Commit b1087fb

Browse files
committed
Removed uid alias for validators executor: KasperskyLab#174
1 parent 50455ba commit b1087fb

2 files changed

Lines changed: 16 additions & 21 deletions

File tree

knp/base-framework/impl/network_validation/executor.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,15 @@
2828
namespace knp::framework::network_validation
2929
{
3030

31-
Executor::ValidatorUID Executor::add_validator(std::string name, PopulationValidator validator)
31+
knp::core::UID Executor::add_validator(std::string name, PopulationValidator validator)
3232
{
3333
knp::core::UID uid;
3434
population_validators_[uid] = {std::move(name), std::move(validator)};
3535
return uid;
3636
}
3737

3838

39-
Executor::ValidatorUID Executor::add_validator(PopulationValidator validator)
39+
knp::core::UID Executor::add_validator(PopulationValidator validator)
4040
{
4141
knp::core::UID uid;
4242
population_validators_[uid] = {
@@ -45,15 +45,15 @@ Executor::ValidatorUID Executor::add_validator(PopulationValidator validator)
4545
}
4646

4747

48-
Executor::ValidatorUID Executor::add_validator(std::string name, ProjectionValidator validator)
48+
knp::core::UID Executor::add_validator(std::string name, ProjectionValidator validator)
4949
{
5050
knp::core::UID uid;
5151
projection_validators_[uid] = {std::move(name), std::move(validator)};
5252
return uid;
5353
}
5454

5555

56-
Executor::ValidatorUID Executor::add_validator(ProjectionValidator validator)
56+
knp::core::UID Executor::add_validator(ProjectionValidator validator)
5757
{
5858
knp::core::UID uid;
5959
projection_validators_[uid] = {
@@ -62,15 +62,15 @@ Executor::ValidatorUID Executor::add_validator(ProjectionValidator validator)
6262
}
6363

6464

65-
Executor::ValidatorUID Executor::add_validator(std::string name, NetworkValidator validator)
65+
knp::core::UID Executor::add_validator(std::string name, NetworkValidator validator)
6666
{
6767
knp::core::UID uid;
6868
network_validators_[uid] = {std::move(name), std::move(validator)};
6969
return uid;
7070
}
7171

7272

73-
Executor::ValidatorUID Executor::add_validator(NetworkValidator validator)
73+
knp::core::UID Executor::add_validator(NetworkValidator validator)
7474
{
7575
knp::core::UID uid;
7676
network_validators_[uid] = {

knp/base-framework/include/knp/framework/network_validation/executor.h

Lines changed: 10 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -41,58 +41,53 @@ namespace knp::framework::network_validation
4141
class KNP_DECLSPEC Executor
4242
{
4343
public:
44-
/**
45-
* @brief Type of validator's UID.
46-
*/
47-
using ValidatorUID = knp::core::UID;
48-
4944
/**
5045
* @brief Add validator to run later.
5146
* @param validator Population validator.
5247
* @param name Validator's display name.
5348
* @return Validator UID.
5449
*/
55-
ValidatorUID add_validator(std::string name, PopulationValidator validator);
50+
knp::core::UID add_validator(std::string name, PopulationValidator validator);
5651

5752
/**
5853
* @brief Add validator to run later.
5954
* @note Automatically generates name.
6055
* @param validator Population validator.
6156
* @return Validator UID.
6257
*/
63-
ValidatorUID add_validator(PopulationValidator validator);
58+
knp::core::UID add_validator(PopulationValidator validator);
6459

6560
/**
6661
* @brief Add validator to run later.
6762
* @param validator Projection validator.
6863
* @param name Validator's display name.
6964
* @return Validator UID.
7065
*/
71-
ValidatorUID add_validator(std::string name, ProjectionValidator validator);
66+
knp::core::UID add_validator(std::string name, ProjectionValidator validator);
7267

7368
/**
7469
* @brief Add validator to run later.
7570
* @note Automatically generates name.
7671
* @param validator Projection validator.
7772
* @return Validator UID.
7873
*/
79-
ValidatorUID add_validator(ProjectionValidator validator);
74+
knp::core::UID add_validator(ProjectionValidator validator);
8075

8176
/**
8277
* @brief Add validator to run later.
8378
* @param validator Network validator.
8479
* @param name Validator's display name.
8580
* @return Validator UID.
8681
*/
87-
ValidatorUID add_validator(std::string name, NetworkValidator validator);
82+
knp::core::UID add_validator(std::string name, NetworkValidator validator);
8883

8984
/**
9085
* @brief Add validator to run later.
9186
* @note Automatically generates name.
9287
* @param validator Network validator.
9388
* @return Validator UID.
9489
*/
95-
ValidatorUID add_validator(NetworkValidator validator);
90+
knp::core::UID add_validator(NetworkValidator validator);
9691

9792
/**
9893
* @brief Validator's report info. Contains name and report.
@@ -103,7 +98,7 @@ class KNP_DECLSPEC Executor
10398
* @brief UID.
10499
*/
105100
// cppcheck-suppress unusedStructMember
106-
ValidatorUID uid_;
101+
knp::core::UID uid_;
107102

108103
/**
109104
* @brief Display name.
@@ -129,11 +124,11 @@ class KNP_DECLSPEC Executor
129124
static void log_report(const Report& report);
130125

131126
// cppcheck-suppress unusedStructMember
132-
std::map<ValidatorUID, std::pair<std::string, PopulationValidator>> population_validators_;
127+
std::map<knp::core::UID, std::pair<std::string, PopulationValidator>> population_validators_;
133128
// cppcheck-suppress unusedStructMember
134-
std::map<ValidatorUID, std::pair<std::string, ProjectionValidator>> projection_validators_;
129+
std::map<knp::core::UID, std::pair<std::string, ProjectionValidator>> projection_validators_;
135130
// cppcheck-suppress unusedStructMember
136-
std::map<ValidatorUID, std::pair<std::string, NetworkValidator>> network_validators_;
131+
std::map<knp::core::UID, std::pair<std::string, NetworkValidator>> network_validators_;
137132
};
138133

139134
} // namespace knp::framework::network_validation

0 commit comments

Comments
 (0)