| … | |
… | |
| 186 | || ob1->speed != ob2->speed |
186 | || ob1->speed != ob2->speed |
| 187 | || ob1->value != ob2->value |
187 | || ob1->value != ob2->value |
| 188 | || ob1->name != ob2->name) |
188 | || ob1->name != ob2->name) |
| 189 | return 0; |
189 | return 0; |
| 190 | |
190 | |
| 191 | /* Do not merge objects if nrof would overflow. First part checks |
191 | /* Do not merge objects if nrof would overflow, assume nrof |
| 192 | * for unsigned overflow (2c), second part checks whether the result |
192 | * is always 0 .. 2**31-1 */ |
| 193 | * would fit into a 32 bit signed int, which is often used to hold |
193 | if (ob1->nrof > 0x7fffffff - ob2->nrof) |
| 194 | * nrof values. |
|
|
| 195 | */ |
|
|
| 196 | if (~ob1->nrof < ob2->nrof || ob1->nrof + ob2->nrof > (1UL << 31)) |
|
|
| 197 | return 0; |
194 | return 0; |
| 198 | |
195 | |
| 199 | /* If the objects have been identified, set the BEEN_APPLIED flag. |
196 | /* If the objects have been identified, set the BEEN_APPLIED flag. |
| 200 | * This is to the comparison of the flags below will be OK. We |
197 | * This is to the comparison of the flags below will be OK. We |
| 201 | * just can't ignore the been applied or identified flags, as they |
198 | * just can't ignore the been applied or identified flags, as they |
| … | |
… | |
| 770 | || (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC)) |
767 | || (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC)) |
| 771 | || (m.move_on | op->move_on ) != m.move_on |
768 | || (m.move_on | op->move_on ) != m.move_on |
| 772 | || (m.move_off | op->move_off ) != m.move_off |
769 | || (m.move_off | op->move_off ) != m.move_off |
| 773 | || (m.move_slow | op->move_slow) != m.move_slow |
770 | || (m.move_slow | op->move_slow) != m.move_slow |
| 774 | /* This isn't perfect, but I don't expect a lot of objects to |
771 | /* This isn't perfect, but I don't expect a lot of objects to |
| 775 | * to have move_allow right now. |
772 | * have move_allow right now. |
| 776 | */ |
773 | */ |
| 777 | || ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block |
774 | || ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block |
| 778 | || 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp) |
775 | || 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp) |
| 779 | m.flags_ = 0; |
776 | m.flags_ = 0; |
| 780 | } |
777 | } |
| … | |
… | |
| 1115 | *(below ? &below->above : &ms.bot) = above; |
1112 | *(below ? &below->above : &ms.bot) = above; |
| 1116 | |
1113 | |
| 1117 | above = 0; |
1114 | above = 0; |
| 1118 | below = 0; |
1115 | below = 0; |
| 1119 | |
1116 | |
|
|
1117 | ms.flags_ = 0; |
|
|
1118 | |
| 1120 | if (map->in_memory == MAP_SAVING) |
1119 | if (map->in_memory == MAP_SAVING) |
| 1121 | return; |
1120 | return; |
| 1122 | |
1121 | |
| 1123 | int check_walk_off = !flag [FLAG_NO_APPLY]; |
1122 | int check_walk_off = !flag [FLAG_NO_APPLY]; |
| 1124 | |
1123 | |
| … | |
… | |
| 1155 | LOG (llevError, "BUG: remove_ob(): name %s, destroyed leaving object\n", tmp->debug_desc ()); |
1154 | LOG (llevError, "BUG: remove_ob(): name %s, destroyed leaving object\n", tmp->debug_desc ()); |
| 1156 | } |
1155 | } |
| 1157 | |
1156 | |
| 1158 | last = tmp; |
1157 | last = tmp; |
| 1159 | } |
1158 | } |
| 1160 | |
|
|
| 1161 | /* last == NULL if there are no objects on this space */ |
|
|
| 1162 | //TODO: this makes little sense, why only update the topmost object? |
|
|
| 1163 | if (!last) |
|
|
| 1164 | map->at (x, y).flags_ = 0; |
|
|
| 1165 | else |
|
|
| 1166 | update_object (last, UP_OBJ_REMOVE); |
|
|
| 1167 | |
1159 | |
| 1168 | if (flag [FLAG_BLOCKSVIEW] || glow_radius) |
1160 | if (flag [FLAG_BLOCKSVIEW] || glow_radius) |
| 1169 | update_all_los (map, x, y); |
1161 | update_all_los (map, x, y); |
| 1170 | } |
1162 | } |
| 1171 | } |
1163 | } |
| … | |
… | |
| 1535 | if (!nr) |
1527 | if (!nr) |
| 1536 | return true; |
1528 | return true; |
| 1537 | |
1529 | |
| 1538 | nr = min (nr, nrof); |
1530 | nr = min (nr, nrof); |
| 1539 | |
1531 | |
|
|
1532 | if (nrof > nr) |
|
|
1533 | { |
| 1540 | nrof -= nr; |
1534 | nrof -= nr; |
| 1541 | |
|
|
| 1542 | if (nrof) |
|
|
| 1543 | { |
|
|
| 1544 | adjust_weight (env, -weight * max (1, nr)); // carrying == 0 |
1535 | adjust_weight (env, -weight * max (1, nr)); // carrying == 0 |
| 1545 | |
1536 | |
| 1546 | if (object *pl = visible_to ()) |
1537 | if (object *pl = visible_to ()) |
| 1547 | esrv_update_item (UPD_NROF, pl, this); |
1538 | esrv_update_item (UPD_NROF, pl, this); |
| 1548 | |
1539 | |
