Add Doxygen comments

Signed-off-by: Darby Johnston <[email protected]>

Author: darbyjohnston

src/opentime/errorStatus.h

@@ -8,8 +8,10 @@
 
 namespace opentime { namespace OPENTIME_VERSION {
 
+/// @brief This struct represents the return status of a function.
 struct ErrorStatus
 {
+    /// @brief This enumeration represents the possible outcomes.
     enum Outcome
     {
         OK = 0,
@@ -21,34 +23,41 @@ struct ErrorStatus
         INVALID_RATE_FOR_DROP_FRAME_TIMECODE,
     };
 
+    /// @brief Construct a new status with no error.
     ErrorStatus()
         : outcome{ OK }
     {}
 
+    /// @brief Construct a new status with the given outcome.
     ErrorStatus(Outcome in_outcome)
         : outcome{ in_outcome }
         , details{ outcome_to_string(in_outcome) }
     {}
 
+    /// @brief Construct a new status with the given outcome and details.
     ErrorStatus(Outcome in_outcome, std::string const& in_details)
         : outcome{ in_outcome }
         , details{ in_details }
     {}
 
-    Outcome     outcome;
+    /// @brief The outcome of the function.
+    Outcome outcome;
+
+    /// @brief A human readable string that provides details about the outcome.
     std::string details;
 
+    ///! @brief Return a human readable string for the given outcome.
     static std::string outcome_to_string(Outcome);
 };
 
-// Check whether the given ErrorStatus is an error.
+///! @brief Check whether the given ErrorStatus is an error.
 constexpr bool
 is_error(const ErrorStatus& es) noexcept
 {
     return ErrorStatus::Outcome::OK != es.outcome;
 }
 
-// Check whether the given ErrorStatus is non-null and an error.
+///! @brief Check whether the given ErrorStatus is non-null and an error.
 constexpr bool
 is_error(const ErrorStatus* es) noexcept
 {

src/opentime/rationalTime.h

@@ -93,7 +93,7 @@ class RationalTime
         return value_rescaled_to(rt._rate);
     }
 
-    /// @brief Returns whether time is almost equal to another time.
+    /// @brief Returns whether this time is almost equal to another time.
     ///
     /// @param other The other time for comparison.
     /// @param delta The tolerance used for the comparison.
@@ -103,10 +103,10 @@ class RationalTime
         return fabs(value_rescaled_to(other._rate) - other._value) <= delta;
     }
 
-    /// @brief Returns whether the value and rate are equal to another time.
+    /// @brief Returns whether this value and rate are exactly equal to another time.
     ///
-    /// This is different from the operator "==" that rescales the time before
-    /// comparison.
+    /// Note that this is different from the equality operator that rescales the time
+    /// before comparison.
     constexpr bool strictly_equal(RationalTime other) const noexcept
     {
         return _value == other._value && _rate == other._rate;

src/opentime/stringPrintf.h

@@ -10,6 +10,7 @@
 
 namespace opentime { namespace OPENTIME_VERSION {
 
+/// @brief This provides printf style functionality for std::string.
 template <typename... Args>
 std::string
 string_printf(char const* format, Args... args)