Fix the ceres.bzl to add missing cc files.

Thanks to nate-thirdwave@ for pointing this out and offering
a fix.

Also add a TODO about an odd loop in covariance_impl.cc which was
revealed as I was testing the bazel build

https: //github.com/ceres-solver/ceres-solver/issues/800
Change-Id: I87d17155ee43ea2a52b8031177d6b3ac5ae1460a

bazel/ceres.bzl

diff --git a/bazel/ceres.bzl b/bazel/ceres.bzl
index fad2754..c76e23a 100644
--- a/bazel/ceres.bzl
+++ b/bazel/ceres.bzl
@@ -32,8 +32,8 @@
     "accelerate_sparse.cc",
     "array_utils.cc",
     "block_evaluate_preparer.cc",
-    "block_jacobian_writer.cc",
     "block_jacobi_preconditioner.cc",
+    "block_jacobian_writer.cc",
     "block_random_access_dense_matrix.cc",
     "block_random_access_diagonal_matrix.cc",
     "block_random_access_matrix.cc",
@@ -53,10 +53,13 @@
     "context_impl.cc",
     "coordinate_descent_minimizer.cc",
     "corrector.cc",
+    "cost_function.cc",
     "covariance.cc",
     "covariance_impl.cc",
+    "cxsparse.cc",
     "dense_cholesky.cc",
     "dense_normal_cholesky_solver.cc",
+    "dense_qr.cc",
     "dense_qr_solver.cc",
     "dense_sparse_matrix.cc",
     "detect_structure.cc",
@@ -65,25 +68,30 @@
     "dynamic_compressed_row_sparse_matrix.cc",
     "dynamic_sparse_normal_cholesky_solver.cc",
     "eigensparse.cc",
+    "evaluation_callback.cc",
     "evaluator.cc",
     "file.cc",
+    "first_order_function.cc",
+    "float_cxsparse.cc",
+    "float_suitesparse.cc",
     "function_sample.cc",
     "gradient_checker.cc",
     "gradient_checking_cost_function.cc",
     "gradient_problem.cc",
     "gradient_problem_solver.cc",
-    "is_close.cc",
     "implicit_schur_complement.cc",
     "inner_product_computer.cc",
+    "is_close.cc",
+    "iteration_callback.cc",
     "iterative_refiner.cc",
     "iterative_schur_complement_solver.cc",
     "levenberg_marquardt_strategy.cc",
     "line_search.cc",
     "line_search_direction.cc",
     "line_search_minimizer.cc",
+    "line_search_preprocessor.cc",
     "linear_least_squares_problems.cc",
     "linear_operator.cc",
-    "line_search_preprocessor.cc",
     "linear_solver.cc",
     "local_parameterization.cc",
     "loss_function.cc",
@@ -92,6 +100,7 @@
     "minimizer.cc",
     "normal_prior.cc",
     "parallel_for_cxx.cc",
+    "parallel_for_nothreads.cc",
     "parallel_for_openmp.cc",
     "parallel_utils.cc",
     "parameter_block_ordering.cc",

internal/ceres/covariance_impl.cc

diff --git a/internal/ceres/covariance_impl.cc b/internal/ceres/covariance_impl.cc
index 15a454f..d6ce69b 100644
--- a/internal/ceres/covariance_impl.cc
+++ b/internal/ceres/covariance_impl.cc
@@ -480,6 +480,9 @@
     // Iterate over the covariance blocks contained in this row block
     // and count the number of columns in this row block.
     int num_col_blocks = 0;
+
+    // TODO(sameeragarwal): num_columns is being computed but not
+    // being used.
     int num_columns = 0;
     for (int j = i; j < covariance_blocks.size(); ++j, ++num_col_blocks) {
       const std::pair<const double*, const double*>& block_pair =