Collect values in a different way

One of the other tests indicates that there actually is supposed to be a
difference between Integer and the values keyword. The latter is needed
to make the evaluate algorithm work. Therefore we should still attempt
to collect values in Indices but failures should not be fatal and
Integer (which we apply no matter what) should do the heavy lifting.

Signed-off-by: Connor Behan <[email protected]>

Author: cbehan

core/properties/Indices.cc

@@ -67,12 +67,22 @@ bool Indices::parse(Kernel& kernel, std::shared_ptr<Ex> ex, keyval_t& keyvals)
 			//std::cerr << "got values keyword " << *(ki->second->name) << std::endl;
 			if(*ki->second->name=="\\sequence") {
 				auto args = std::make_shared<cadabra::Ex>(ki->second);
-				kernel.inject_property(new Integer(), ex, args);
+				auto prop = new Integer();
+				kernel.inject_property(prop, ex, args);
+
+				if (prop->from.is_integer() && prop->to.is_integer()) {
+					if (prop->difference.to_integer() < 100) {
+						for (int i=prop->from.to_integer(); i<=prop->to.to_integer(); ++i) {
+							values.push_back(Ex(i));
+							}
+						}
+					}
+
 				++ki;
 				continue;
 				}
-			collect_index_values(ki->second);
 
+			collect_index_values(ki->second);
 			// If all values are indices, add an `Integer' property for the object,
 			// listing these integers.
 			bool is_number=true;