Skip to content

Commit f6a1d95

Browse files
jmirabelolivier-stasse
authored andcommitted
Fix minor bugs.
1 parent a384a50 commit f6a1d95

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

include/sot/core/variadic-op.hh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,6 @@ namespace dynamicgraph {
6767
{
6868
for (std::size_t i = 0; i < signalsIN.size(); ++i) {
6969
_removeSignal (i);
70-
delete signalsIN[i];
7170
}
7271
};
7372

@@ -98,7 +97,7 @@ namespace dynamicgraph {
9897
void removeSignal ()
9998
{
10099
assert (signalsIN.size()>0);
101-
_removeSignal (signalsIN().size()-1);
100+
_removeSignal (signalsIN.size()-1);
102101
// names.pop_back();
103102
signalsIN.pop_back();
104103
}

src/matrix/operator.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -961,7 +961,7 @@ namespace dynamicgraph {
961961
AdderVariadic () : coeffs () {}
962962
void operator()( const std::vector<const T*>& vs, T& res ) const
963963
{
964-
assert (vs.size() == coeffs.size());
964+
assert (vs.size() == (std::size_t)coeffs.size());
965965
if (vs.size() == 0) return;
966966
res = coeffs[0]*(*vs[0]);
967967
for (std::size_t i = 1; i < vs.size(); ++i)
@@ -1088,7 +1088,6 @@ namespace dynamicgraph {
10881088
// TODO computation could be optimized with lazy evaluation of the
10891089
// signals. When the output result is know, the remaining signals are
10901090
// not computed.
1091-
assert (vs.size() == coeffs.size());
10921091
if (vs.size() == 0) return;
10931092
res = *vs[0];
10941093
for (std::size_t i = 1; i < vs.size(); ++i)

0 commit comments

Comments
 (0)