gtborg / gtsam / issues / #449 - Duplicate FactorIndices class in gtsam.h — Bitbucket

Issue #449 closed

Varun Agrawal created an issue 2019-04-09

Commit 285ebd introduced another definition of class FactorIndices when one already existed on line 2244.

This is breaking the build for the MATLAB wrapper.

Comments (3)

Varun Agrawal reporter
@jlblancoc I removed the definition on line 2244 of gtsam.h and it compilation continues without issues.
- 2019-04-09T22:28:15+00:00
José Luis Blanco-Claraco
Ooops... I guess neither CI nor myself built Matlab wrappers so it went unnoticed. Please, go ahead and do the commit and/or PR yourself. Thanks for the quick fix!
- 2019-04-09T22:45:33+00:00
Frank Dellaert
- changed status to closed
Merged in bug-fix/factoindices-duplicate (pull request #411) Close issue ~~#449~~ Remove duplicate declaration of FactorIndices class

→ <<cset fe0c951e23b8>>
- 2019-04-10T03:11:48+00:00
Log in to comment

Assignee: José Luis Blanco-Claraco

Type: bug

Priority: critical

Status: closed

Milestone: GTSAM Bugs

Version: 4.0.0

Votes: 0

Watchers: 2

Jira: the preferred issue tracker for Bitbucket. Join the team!