You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ac...@apache.org on 2017/09/07 14:32:28 UTC
[2/2] qpid-proton git commit: PROTON-1576: c++: remove link_namer
from public API
PROTON-1576: c++: remove link_namer from public API
It is an internal detail of our container implementation, does not need to be public.
Project: http://git-wip-us.apache.org/repos/asf/qpid-proton/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-proton/commit/c870da9f
Tree: http://git-wip-us.apache.org/repos/asf/qpid-proton/tree/c870da9f
Diff: http://git-wip-us.apache.org/repos/asf/qpid-proton/diff/c870da9f
Branch: refs/heads/master
Commit: c870da9fa02bb884322ca893e22c878b2bc3edca
Parents: c99ab4a
Author: Alan Conway <ac...@redhat.com>
Authored: Thu Sep 7 10:30:11 2017 -0400
Committer: Alan Conway <ac...@redhat.com>
Committed: Thu Sep 7 10:30:11 2017 -0400
----------------------------------------------------------------------
.../cpp/include/proton/io/link_namer.hpp | 51 --------------------
.../bindings/cpp/src/connection_driver_test.cpp | 2 +-
.../bindings/cpp/src/include/link_namer.hpp | 51 ++++++++++++++++++++
proton-c/bindings/cpp/src/io/link_namer.cpp | 2 +-
proton-c/bindings/cpp/src/session.cpp | 2 +-
5 files changed, 54 insertions(+), 54 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/qpid-proton/blob/c870da9f/proton-c/bindings/cpp/include/proton/io/link_namer.hpp
----------------------------------------------------------------------
diff --git a/proton-c/bindings/cpp/include/proton/io/link_namer.hpp b/proton-c/bindings/cpp/include/proton/io/link_namer.hpp
deleted file mode 100644
index e6d84ee..0000000
--- a/proton-c/bindings/cpp/include/proton/io/link_namer.hpp
+++ /dev/null
@@ -1,51 +0,0 @@
-#ifndef PROTON_IO_LINK_NAMER_HPP
-#define PROTON_IO_LINK_NAMER_HPP
-
-/*
- *
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- *
- */
-
-#include "../internal/export.hpp"
-#include <string>
-
-namespace proton {
-
-class connection;
-
-namespace io {
-
-/// **Unsettled API** - Generate default link names that are unique
-/// within a container. base_container provides a default
-/// implementation.
-class link_namer {
- public:
- virtual ~link_namer() {}
-
- /// Generate a unique link name.
- virtual std::string link_name() = 0;
-};
-
-/// **Unsettled API** - Set the link_namer to use on a connection.
-PN_CPP_EXTERN void set_link_namer(connection&, link_namer&);
-
-} // io
-} // proton
-
-#endif // PROTON_IO_LINK_NAMER_HPP
http://git-wip-us.apache.org/repos/asf/qpid-proton/blob/c870da9f/proton-c/bindings/cpp/src/connection_driver_test.cpp
----------------------------------------------------------------------
diff --git a/proton-c/bindings/cpp/src/connection_driver_test.cpp b/proton-c/bindings/cpp/src/connection_driver_test.cpp
index d174454..18d4f6e 100644
--- a/proton-c/bindings/cpp/src/connection_driver_test.cpp
+++ b/proton-c/bindings/cpp/src/connection_driver_test.cpp
@@ -20,11 +20,11 @@
#include "test_bits.hpp"
#include "proton_bits.hpp"
+#include "link_namer.hpp"
#include "proton/container.hpp"
#include "proton/connection.hpp"
#include "proton/io/connection_driver.hpp"
-#include "proton/io/link_namer.hpp"
#include "proton/link.hpp"
#include "proton/message.hpp"
#include "proton/messaging_handler.hpp"
http://git-wip-us.apache.org/repos/asf/qpid-proton/blob/c870da9f/proton-c/bindings/cpp/src/include/link_namer.hpp
----------------------------------------------------------------------
diff --git a/proton-c/bindings/cpp/src/include/link_namer.hpp b/proton-c/bindings/cpp/src/include/link_namer.hpp
new file mode 100644
index 0000000..d107728
--- /dev/null
+++ b/proton-c/bindings/cpp/src/include/link_namer.hpp
@@ -0,0 +1,51 @@
+#ifndef PROTON_IO_LINK_NAMER_HPP
+#define PROTON_IO_LINK_NAMER_HPP
+
+/*
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ *
+ */
+
+#include "proton/internal/export.hpp"
+#include <string>
+
+namespace proton {
+
+class connection;
+
+namespace io {
+
+/// **Unsettled API** - Generate default link names that are unique
+/// within a container. base_container provides a default
+/// implementation.
+class link_namer {
+ public:
+ virtual ~link_namer() {}
+
+ /// Generate a unique link name.
+ virtual std::string link_name() = 0;
+};
+
+/// **Unsettled API** - Set the link_namer to use on a connection.
+PN_CPP_EXTERN void set_link_namer(connection&, link_namer&);
+
+} // io
+} // proton
+
+#endif // PROTON_IO_LINK_NAMER_HPP
http://git-wip-us.apache.org/repos/asf/qpid-proton/blob/c870da9f/proton-c/bindings/cpp/src/io/link_namer.cpp
----------------------------------------------------------------------
diff --git a/proton-c/bindings/cpp/src/io/link_namer.cpp b/proton-c/bindings/cpp/src/io/link_namer.cpp
index 2ac5402..c8c7c8c 100644
--- a/proton-c/bindings/cpp/src/io/link_namer.cpp
+++ b/proton-c/bindings/cpp/src/io/link_namer.cpp
@@ -17,7 +17,7 @@
* under the License.
*/
-#include "proton/io/link_namer.hpp"
+#include "link_namer.hpp"
#include "proton/connection.hpp"
http://git-wip-us.apache.org/repos/asf/qpid-proton/blob/c870da9f/proton-c/bindings/cpp/src/session.cpp
----------------------------------------------------------------------
diff --git a/proton-c/bindings/cpp/src/session.cpp b/proton-c/bindings/cpp/src/session.cpp
index 4c3932b..d2fb401 100644
--- a/proton-c/bindings/cpp/src/session.cpp
+++ b/proton-c/bindings/cpp/src/session.cpp
@@ -21,12 +21,12 @@
#include "proton/session.hpp"
#include "proton/connection.hpp"
-#include "proton/io/link_namer.hpp"
#include "proton/receiver_options.hpp"
#include "proton/sender_options.hpp"
#include "proton/session_options.hpp"
#include "contexts.hpp"
+#include "link_namer.hpp"
#include "proton_bits.hpp"
#include <proton/connection.h>
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org