https://smaugmuds.afkmods.com/index.php?a=rssfeed&t=4880 The largest Smaug community resource site. - en SmaugMuds 60 https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23848#p23848 [quote=Leia]Your mud "groups objects" with the group_object() function? If so, it's counting only the objects that have different affects. [code]short inventory_total(CHAR_DATA *ch) { OBJ_DATA *findobj, *findobj_next; short total = 0; for(findobj = ch->first_carrying; findobj != NULL; findobj = findobj_next) { findobj_next = findobj->next_content; total += findobj->count; <-------------------- That should solve it. https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23848#p23848 Fri, 13 Apr 2018 19:19:26 CDT Coding nobody@example.com (Vladaar) https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23847#p23847 That's got to be it. Thanks! https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23847#p23847 Fri, 13 Apr 2018 15:28:57 CDT Coding nobody@example.com (Vladaar) https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23846#p23846 Your mud "groups objects" with the group_object() function? If so, it's counting only the objects that have different affects. [code]short inventory_total(CHAR_DATA *ch) { OBJ_DATA *findobj, *findobj_next; short total = 0; for(findobj = ch->first_carrying; findobj != NULL; findobj = findobj_next) { findobj_next = findobj->next_content; total += findobj->count; <-------------------- That should solve it. } bug( " https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23846#p23846 Fri, 13 Apr 2018 14:49:45 CDT Coding nobody@example.com (Leia) https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23845#p23845 Leia that solved the problem with it not putting the proper amount in the bundle, but now my total_inventory is jacked up. This is my current working total_inventory function [code] short inventory_total(CHAR_DATA *ch) { OBJ_DATA *findobj, *findobj_next; short total = 0; for(findobj = ch->first_carrying; findobj != NULL; findobj = findobj_next) { findobj_next = findobj->next_content; total++; } bug( "%s total %d" https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23845#p23845 Fri, 13 Apr 2018 12:55:43 CDT Coding nobody@example.com (Vladaar) https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23844#p23844 Try this: [code] OBJ_DATA *check_prev; while ( count2 != -1 && count != number ) { for(check = ch->last_carrying; check; check = check_prev) { check_prev = check->prev_content; if ( check->pIndexData->vnum == jackpot && count < number ) { if ( ch ) count++; separate_obj(check); obj_from_char(check); obj_to_obj( check, bundle ); } } count2--; }[/code] https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23844#p23844 Fri, 13 Apr 2018 11:17:25 CDT Coding nobody@example.com (Leia) https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23843#p23843 Hey guys I need a fresh set of eyes to look at this. I am missing something simple I guess. For some reason my obj_from_char call is having NULL ch a few times in the loop and screwing up my obj_to_obj call numbers. What the intent of this code was, is to bundle up crafted items that players do, so that I can have a player craft 20 odd things for a quest mob, and carry it as 1 object giving a bundle of 20 crafted things to the mob to activate the act_prog. [code] /* too many damn loops https://smaugmuds.afkmods.com/topic/objfromchar-null-ch-issues-4880/&p=23843#p23843 Thu, 12 Apr 2018 18:47:45 CDT Coding nobody@example.com (Vladaar)