Remove std::vector constructor to FQName.

This is really just to construct the package itself, but
it's pretty confusing to read in context and is only used
once. Since we are moving FQName to not have an 'isValid'
field, this is one less thing to worry about.

Bug: 72125849
Test: hidl-gen's run_all_host_tests.sh
Change-Id: Ie5500e0d0c4ce0c1548562cd4a96236e3223e1ec

2 files changed, 0 insertions, 9 deletions

diff --git a/utils/FQName.cpp b/utils/FQName.cpp
index bb62244f..1fa61cfd 100644
--- a/utils/FQName.cpp
+++ b/utils/FQName.cpp
@@ -91,12 +91,6 @@ FQName::FQName(const FQName& other)
       mValueName(other.mValueName) {
 }
 
-FQName::FQName(const std::vector<std::string> &names)
-    : mValid(false),
-      mIsIdentifier(false) {
-    setTo(StringHelper::JoinStrings(names, "."));
-}
-
 bool FQName::isValid() const {
     return mValid;
 }
diff --git a/utils/include/hidl-util/FQName.h b/utils/include/hidl-util/FQName.h
index 3cfe61ac..d5bfc33d 100644
--- a/utils/include/hidl-util/FQName.h
+++ b/utils/include/hidl-util/FQName.h
@@ -32,9 +32,6 @@ struct FQName {
            const std::string &name,
            const std::string &valueName = "");
 
-    // a synonym to FQName(names.join("."))
-    FQName(const std::vector<std::string> &names);
-
     FQName(const FQName& other);
 
     bool isValid() const;