Skip to content

Commit

Permalink
fix some build problems
Browse files Browse the repository at this point in the history
Specifically: in elements.h the translate:: is redundant since we
are already in the translate namespace, and that causes a warning.

In the other file we -> into a Symbol_sp even though symbol.h was
not directly included. It was included transiently but I rearranged
that away in Clasp in a branch.
  • Loading branch information
Bike committed Oct 26, 2024
1 parent 3855d10 commit e283498
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions include/cando/chem/alias.h
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ This is an open source license for the CANDO software from Temple University, bu
#include <vector>
#include <set>
#include <clasp/core/common.h>
#include <clasp/core/symbol.h>
#include <cando/geom/vector3.h>


Expand Down
4 changes: 2 additions & 2 deletions include/cando/chem/elements.h
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@ class ElementsInfo_O : public core::CxxObject_O
namespace translate
{
template <>
struct translate::from_object<chem::Element>
struct from_object<chem::Element>
{
typedef chem::Element ExpectedType;
typedef chem::Element DeclareType;
Expand Down Expand Up @@ -171,7 +171,7 @@ namespace translate
};

template <>
struct translate::from_object<chem::Hybridization>
struct from_object<chem::Hybridization>
{
typedef chem::Hybridization ExpectedType;
typedef chem::Hybridization DeclareType;
Expand Down

0 comments on commit e283498

Please sign in to comment.