From f9e8497d5c2415eb11c3c7027db8ee8323271e89 Mon Sep 17 00:00:00 2001
From: Alex Cunningham <alexgcunningham@gmail.com>
Date: Fri, 17 Jun 2011 16:53:11 +0000
Subject: [PATCH] Removed fixmes near includes

---
 gtsam/inference/Makefile.am   | 3 ++-
 gtsam/inference/inference.cpp | 2 --
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/gtsam/inference/Makefile.am b/gtsam/inference/Makefile.am
index 6c8781498..6bd273b06 100644
--- a/gtsam/inference/Makefile.am
+++ b/gtsam/inference/Makefile.am
@@ -49,7 +49,8 @@ check_PROGRAMS += tests/testJunctionTree
 # The headers are installed in $(includedir)/gtsam: 
 #----------------------------------------------------------------------------------------------------
 
-#FIXME: inclusion of ccolamd is annoying
+# CCOLAMD include flags are needed due to the bad include paths within the library
+# but will not be exposed to users.  
 ccolamd_inc = -I$(top_srcdir)/gtsam/3rdparty/CCOLAMD/Include -I$(top_srcdir)/gtsam/3rdparty/UFconfig
 
 headers += $(sources:.cpp=.h)
diff --git a/gtsam/inference/inference.cpp b/gtsam/inference/inference.cpp
index 24eea5deb..d30ea79c4 100644
--- a/gtsam/inference/inference.cpp
+++ b/gtsam/inference/inference.cpp
@@ -23,8 +23,6 @@
 #include <iostream>
 #include <vector>
 
-// FIXME: handle include path better
-//#include <gtsam/3rdparty/CCOLAMD/Include/ccolamd.h>
 #include <ccolamd.h>
 
 using namespace std;