[datasketches-cpp] branch master updated: Include header implicitly used

[al...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

alsay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/datasketches-cpp.git


The following commit(s) were added to refs/heads/master by this push:
     new de1bb59  Include header implicitly used
     new d924cc6  Merge pull request #326 from jbapple/theta-includes
de1bb59 is described below

commit de1bb593205c4cb1850a536df0504c7d8c9197d3
Author: Jim Apple <jb...@example.com>
AuthorDate: Mon Jan 9 10:08:35 2023 -0800

    Include header implicitly used
    
    theta_helpers.hpp uses theta_constants::MAX_THETA, which is declared
    in theta_constants.hpp. This patch #includes it as well as
    alphabetizes the standard headers.
---
 theta/include/theta_helpers.hpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/theta/include/theta_helpers.hpp b/theta/include/theta_helpers.hpp
index 76c1c23..cbdebb4 100644
--- a/theta/include/theta_helpers.hpp
+++ b/theta/include/theta_helpers.hpp
@@ -20,8 +20,10 @@
 #ifndef THETA_HELPERS_HPP_
 #define THETA_HELPERS_HPP_
 
-#include <string>
 #include <stdexcept>
+#include <string>
+
+#include "theta_constants.hpp"
 
 namespace datasketches {
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datasketches.apache.org
For additional commands, e-mail: commits-help@datasketches.apache.org