Fix #114428: clamp setting object active_material_index

There were multiple reports with objects having many empty material
slots.
The underlying reason for this is the behavior of adding/assigning
materials [which makes room in the form of empty material slots based on
the current `active_material_index` -- which atm. can be set to
arbitrary values]. So just e.g. setting this to 100 in a fresh file and
assigning a material would create 99 empty slots.

to resolve, now clamp to the existing number of material slots.

NOTE: there is already a range function defined, but this actually only
kicks in from the animation system (so clamping would take place there),
so clamping is expected to happen in the set functions (there is also a
related comment in `RNA_property_int_set`)

@@ -1123,1 +1123,3 @@
-  ob->actcol = value + 1;
+
+  ob->actcol = min_ii(ob->totcol, value + 1);
+  ob->actcol = max_ii(ob->actcol, 0);

@@ -1121,3 +1121,3 @@
 {
   Object *ob = reinterpret_cast<Object *>(ptr->owner_id);
-  ob->actcol = value + 1;
+

@@ -1121,2 +1121,4 @@
 {
   Object *ob = reinterpret_cast<Object *>(ptr->owner_id);
+
+  value = std::clamp(value, 0, ob->totcol - 1);

@@ -1121,2 +1121,4 @@
 {
   Object *ob = reinterpret_cast<Object *>(ptr->owner_id);
+
+  value = (ob->totcol > 1) ? std::clamp(value, 0, ob->totcol - 1) : 0;