Comment cleanup (#395)

maint/external-prereq-packages.ubuntu-16.04.txt

-# Required to build the main components of NGraph-C++
+# Required to build the main components of nGraph-C++
 build-essential
 cmake
 git
 
-# Required to build the documentation of NGraph-C++
+# Required to build the documentation of nGraph-C++
 doxygen
 sphinx-build
 

src/ngraph/ops/broadcast.hpp

@@ -70,7 +70,7 @@ namespace ngraph
                 return std::make_shared<Broadcast>(new_args.at(0), m_shape, m_broadcast_axes);
             }
 
-            /// \return An set containing the indices of the broadcast axes (0-based).
+            /// \return A set containing the indices of the broadcast axes (0-based).
             const AxisSet& get_broadcast_axes() const { return m_broadcast_axes; }
             const Shape& get_broadcast_shape() const { return m_shape; }
             bool is_functionally_identical(const Node&) const override;

src/ngraph/runtime/call_frame.hpp

@@ -53,11 +53,10 @@ namespace ngraph
             virtual ~CallFrame() {}
             /// @brief Invoke the function with values matching the signature of the function.
             ///
-            /// Tuples will be expanded into their tensor views to build the call frame.
             virtual void call(const std::vector<std::shared_ptr<runtime::TensorView>>& inputs,
                               const std::vector<std::shared_ptr<runtime::TensorView>>& outputs) = 0;
 
-            /// @brief Invoke the function with tuples pre-expanded to their underlying tensor views.
+            /// @brief Invoke the function
             virtual void tensor_call(const TensorViewPtrs& inputs,
                                      const TensorViewPtrs& outputs) = 0;
 

src/ngraph/serializer.cpp

@@ -73,9 +73,8 @@ static std::shared_ptr<ngraph::Function>
 static json write(const ngraph::Function&);
 static json write(const ngraph::Node&);
 
-// This stupidity is caused by the fact that we do not pass element types
-// by value but by reference even though they can be compared. There is no reason to pass
-// them by reference EVERYWERE but here we are...
+// There should be a map from element type names to element types so deserialization can
+// find the singletons and serialization can serialize by name.
 static const element::Type& to_ref(const element::Type& t)
 {
     if (t == element::boolean)