ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/deliantra/server/socket/request.C
(Generate patch)

Comparing deliantra/server/socket/request.C (file contents):
Revision 1.27 by root, Wed Dec 13 18:08:02 2006 UTC vs.
Revision 1.31 by root, Thu Dec 14 00:23:59 2006 UTC

682 */ 682 */
683static void 683static void
684SendSmooth (NewSocket * ns, uint16 face) 684SendSmooth (NewSocket * ns, uint16 face)
685{ 685{
686 uint16 smoothface; 686 uint16 smoothface;
687 unsigned char reply[MAX_BUF];
688 SockList sl;
689 687
690 /* If we can't find a face, return and set it so we won't try to send this 688 /* If we can't find a face, return and set it so we won't try to send this
691 * again. 689 * again.
692 */ 690 */
693 if ((!FindSmooth (face, &smoothface)) && (!FindSmooth (smooth_face->number, &smoothface))) 691 if ((!FindSmooth (face, &smoothface)) && (!FindSmooth (smooth_face->number, &smoothface)))
701 if (!(ns->faces_sent[smoothface] & NS_FACESENT_FACE)) 699 if (!(ns->faces_sent[smoothface] & NS_FACESENT_FACE))
702 esrv_send_face (ns, smoothface, 0); 700 esrv_send_face (ns, smoothface, 0);
703 701
704 ns->faces_sent[face] |= NS_FACESENT_SMOOTH; 702 ns->faces_sent[face] |= NS_FACESENT_SMOOTH;
705 703
706 sl.buf = reply; 704 SockList sl (MAXSOCKBUF);
707 strcpy ((char *) sl.buf, "smooth ");
708 sl.len = strlen ((char *) sl.buf);
709 705
710 sl << uint16 (face) << uint16 (smoothface); 706 sl << "smooth "
707 << uint16 (face)
708 << uint16 (smoothface);
711 709
712 Send_With_Handling (ns, &sl); 710 Send_With_Handling (ns, &sl);
711 sl.free ();
713} 712}
714 713
715 /** 714 /**
716 * Tells client the picture it has to use 715 * Tells client the picture it has to use
717 * to smooth a picture number given as argument. 716 * to smooth a picture number given as argument.
723 722
724 facenbr = atoi (buf); 723 facenbr = atoi (buf);
725 SendSmooth (ns, facenbr); 724 SendSmooth (ns, facenbr);
726} 725}
727 726
728
729
730
731
732/** 727/**
733 * This handles the general commands from the client (ie, north, fire, cast, 728 * This handles the general commands from the client (ie, north, fire, cast,
734 * etc.) 729 * etc.)
735 */ 730 */
736void 731void
789 */ 784 */
790void 785void
791NewPlayerCmd (uint8 * buf, int len, player *pl) 786NewPlayerCmd (uint8 * buf, int len, player *pl)
792{ 787{
793 int time, repeat; 788 int time, repeat;
789 char command[MAX_BUF];
794 short packet; 790 short packet;
795 unsigned char command[MAX_BUF];
796 SockList sl;
797 791
798 if (len < 7) 792 if (len < 7)
799 { 793 {
800 LOG (llevDebug, "Corrupt ncom command <%s> not long enough - discarding\n", buf); 794 LOG (llevDebug, "Corrupt ncom command <%s> not long enough - discarding\n", buf);
801 return; 795 return;
802 } 796 }
803 797
804 packet = GetShort_String (buf); 798 packet = net_uint16 (buf);
805 repeat = GetInt_String (buf + 2); 799 repeat = net_uint32 (buf + 2);
800
806 /* -1 is special - no repeat, but don't update */ 801 /* -1 is special - no repeat, but don't update */
807 if (repeat != -1) 802 if (repeat != -1)
808 {
809 pl->count = repeat; 803 pl->count = repeat;
810 } 804
811 if ((len - 4) >= MAX_BUF) 805 if ((len - 4) >= MAX_BUF)
812 len = MAX_BUF - 5; 806 len = MAX_BUF - 5;
813 807
814 strncpy ((char *) command, (char *) buf + 6, len - 4); 808 strncpy ((char *) command, (char *) buf + 6, len - 4);
815 command[len - 4] = '\0'; 809 command[len - 4] = '\0';
825 return; 819 return;
826 } 820 }
827 821
828 /* This should not happen anymore. */ 822 /* This should not happen anymore. */
829 if (pl->ob->speed_left < -1.0) 823 if (pl->ob->speed_left < -1.0)
830 {
831 LOG (llevError, "Player has negative time - shouldn't do command.\n"); 824 LOG (llevError, "Player has negative time - shouldn't do command.\n");
832 } 825
833 /* In c_new.c */ 826 /* In c_new.c */
834 execute_newserver_command (pl->ob, (char *) command); 827 execute_newserver_command (pl->ob, (char *) command);
835 /* Perhaps something better should be done with a left over count. 828 /* Perhaps something better should be done with a left over count.
836 * Cleaning up the input should probably be done first - all actions 829 * Cleaning up the input should probably be done first - all actions
837 * for the command that issued the count should be done before any other 830 * for the command that issued the count should be done before any other
838 * commands. 831 * commands.
839 */ 832 */
840 pl->count = 0; 833 pl->count = 0;
841 834
842 /* Send confirmation of command execution now */
843 sl.buf = command;
844 strcpy ((char *) sl.buf, "comc ");
845 sl.len = 5;
846
847 if (FABS (pl->ob->speed) < 0.001) 835 if (FABS (pl->ob->speed) < 0.001)
848 time = MAX_TIME * 100; 836 time = MAX_TIME * 100;
849 else 837 else
850 time = (int) (MAX_TIME / FABS (pl->ob->speed)); 838 time = (int) (MAX_TIME / FABS (pl->ob->speed));
851 839
840 /* Send confirmation of command execution now */
841
842 SockList sl (MAXSOCKBUF);
852 sl << uint16 (packet) << uint32 (time); 843 sl << "comc " << uint16 (packet) << uint32 (time);
853
854 Send_With_Handling (&pl->socket, &sl); 844 Send_With_Handling (&pl->socket, &sl);
845 sl.free ();
855} 846}
856 847
857 848
858/** This is a reply to a previous query. */ 849/** This is a reply to a previous query. */
859void 850void
971 /* these are old dxclients */ 962 /* these are old dxclients */
972 /* Version 1024 added support for singular + plural name values - 963 /* Version 1024 added support for singular + plural name values -
973 * requiing this minimal value reduces complexity of that code, and it 964 * requiing this minimal value reduces complexity of that code, and it
974 * has been around for a long time. 965 * has been around for a long time.
975 */ 966 */
976 if (!strcmp (" CF DX CLIENT", cp) || ns->sc_version < 1024) 967 if (ns->sc_version < 1026)
977 { 968 {
978 sprintf (version_warning, "drawinfo %d %s", NDI_RED, 969 sprintf (version_warning, "drawinfo %d %s", NDI_RED,
979 "**** VERSION WARNING ****\n**** CLIENT IS TOO OLD!! UPDATE THE CLIENT!! ****"); 970 "**** VERSION WARNING ****\n**** CLIENT IS TOO OLD!! UPDATE THE CLIENT!! ****");
980 Write_String_To_Socket (ns, version_warning, strlen (version_warning)); 971 Write_String_To_Socket (ns, version_warning, strlen (version_warning));
981 } 972 }
1062 Write_String_To_Socket (ns, buf, strlen (buf)); 1053 Write_String_To_Socket (ns, buf, strlen (buf));
1063} 1054}
1064 1055
1065#define AddIfInt64(Old,New,Type) if (Old != New) {\ 1056#define AddIfInt64(Old,New,Type) if (Old != New) {\
1066 Old = New; \ 1057 Old = New; \
1067 SockList_AddChar(&sl, Type); \ 1058 sl << uint8 (Type) << uint64 (New); \
1068 SockList_AddInt64(&sl, New); \
1069 } 1059 }
1070 1060
1071#define AddIfInt(Old,New,Type) if (Old != New) {\ 1061#define AddIfInt(Old,New,Type) if (Old != New) {\
1072 Old = New; \ 1062 Old = New; \
1073 SockList_AddChar(&sl, Type); \ 1063 sl << uint8 (Type) << uint32 (New); \
1074 SockList_AddInt(&sl, New); \
1075 } 1064 }
1076 1065
1077#define AddIfShort(Old,New,Type) if (Old != New) {\ 1066#define AddIfShort(Old,New,Type) if (Old != New) {\
1078 Old = New; \ 1067 Old = New; \
1079 SockList_AddChar(&sl, Type); \ 1068 sl << uint8 (Type) << uint16 (New); \
1080 SockList_AddShort(&sl, New); \
1081 } 1069 }
1082 1070
1083#define AddIfFloat(Old,New,Type) if (Old != New) {\ 1071#define AddIfFloat(Old,New,Type) if (Old != New) {\
1084 Old = New; \ 1072 Old = New; \
1085 SockList_AddChar(&sl, Type); \ 1073 sl << uint8 (Type) << uint32 (New*FLOAT_MULTI); \
1086 SockList_AddInt(&sl,(long)(New*FLOAT_MULTI));\
1087 } 1074 }
1088 1075
1089#define AddIfString(Old,New,Type) if (Old == NULL || strcmp(Old,New)) {\ 1076#define AddIfString(Old,New,Type) if (Old == NULL || strcmp(Old,New)) {\
1090 if (Old) free(Old);\
1091 Old = strdup_local(New);\ 1077 free(Old); Old = strdup (New);\
1092 SockList_AddChar(&sl, Type); \ 1078 sl << uint8 (Type) << data8 (New); \
1093 SockList_AddChar(&sl, ( char )strlen(New)); \
1094 strcpy((char*)sl.buf + sl.len, New); \
1095 sl.len += strlen(New); \
1096 } 1079 }
1097 1080
1098/** 1081/**
1099 * Sends a statistics update. We look at the old values, 1082 * Sends a statistics update. We look at the old values,
1100 * and only send what has changed. Stat mapping values are in newclient.h 1083 * and only send what has changed. Stat mapping values are in newclient.h
1102 * commands for now. 1085 * commands for now.
1103 */ 1086 */
1104void 1087void
1105esrv_update_stats (player *pl) 1088esrv_update_stats (player *pl)
1106{ 1089{
1107 SockList sl;
1108 char buf[MAX_BUF]; 1090 char buf[MAX_BUF];
1109 uint16 flags; 1091 uint16 flags;
1110 1092
1111 sl.buf = (unsigned char *) malloc (MAXSOCKBUF); 1093 SockList sl (MAXSOCKBUF);
1112 strcpy ((char *) sl.buf, "stats "); 1094 sl << "stats ";
1113 sl.len = strlen ((char *) sl.buf);
1114 1095
1115 if (pl->ob != NULL) 1096 if (pl->ob != NULL)
1116 { 1097 {
1117 AddIfShort (pl->last_stats.hp, pl->ob->stats.hp, CS_STAT_HP); 1098 AddIfShort (pl->last_stats.hp, pl->ob->stats.hp, CS_STAT_HP);
1118 AddIfShort (pl->last_stats.maxhp, pl->ob->stats.maxhp, CS_STAT_MAXHP); 1099 AddIfShort (pl->last_stats.maxhp, pl->ob->stats.maxhp, CS_STAT_MAXHP);
1135 1116
1136 for (s = 0; s < NUM_SKILLS; s++) 1117 for (s = 0; s < NUM_SKILLS; s++)
1137 { 1118 {
1138 if (pl->last_skill_ob[s] && pl->last_skill_exp[s] != pl->last_skill_ob[s]->stats.exp) 1119 if (pl->last_skill_ob[s] && pl->last_skill_exp[s] != pl->last_skill_ob[s]->stats.exp)
1139 { 1120 {
1140
1141 /* Always send along the level if exp changes. This is only 1121 /* Always send along the level if exp changes. This is only
1142 * 1 extra byte, but keeps processing simpler. 1122 * 1 extra byte, but keeps processing simpler.
1143 */ 1123 */
1144 SockList_AddChar (&sl, (char) (s + CS_STAT_SKILLINFO)); 1124 sl << uint8 (s + CS_STAT_SKILLINFO)
1145 SockList_AddChar (&sl, (char) pl->last_skill_ob[s]->level); 1125 << uint8 (pl->last_skill_ob[s]->level)
1146 SockList_AddInt64 (&sl, pl->last_skill_ob[s]->stats.exp); 1126 << uint64 (pl->last_skill_ob[s]->stats.exp);
1127
1147 pl->last_skill_exp[s] = pl->last_skill_ob[s]->stats.exp; 1128 pl->last_skill_exp[s] = pl->last_skill_ob[s]->stats.exp;
1148 } 1129 }
1149 } 1130 }
1150 } 1131 }
1151 1132
1207 LOG (llevDebug, "Sending stats command, %d bytes long.\n", sl.len); 1188 LOG (llevDebug, "Sending stats command, %d bytes long.\n", sl.len);
1208#endif 1189#endif
1209 Send_With_Handling (&pl->socket, &sl); 1190 Send_With_Handling (&pl->socket, &sl);
1210 } 1191 }
1211 1192
1212 free (sl.buf); 1193 sl.free ();
1213} 1194}
1214 1195
1215/** 1196/**
1216 * Tells the client that here is a player it should start using. 1197 * Tells the client that here is a player it should start using.
1217 */ 1198 */
1218void 1199void
1219esrv_new_player (player *pl, uint32 weight) 1200esrv_new_player (player *pl, uint32 weight)
1220{ 1201{
1221 SockList sl;
1222
1223 pl->last_weight = weight; 1202 pl->last_weight = weight;
1224 1203
1225 sl.buf = (unsigned char *) malloc (MAXSOCKBUF); 1204 SockList sl (MAXSOCKBUF);
1205 sl << "player ";
1226 1206
1227 strcpy ((char *) sl.buf, "player "); 1207 sl << uint32 (pl->ob->count)
1228 sl.len = strlen ((char *) sl.buf); 1208 << uint32 (weight)
1229 SockList_AddInt (&sl, pl->ob->count); 1209 << uint32 (pl->ob->face->number)
1230 SockList_AddInt (&sl, weight); 1210 << data8 (pl->ob->name);
1231 SockList_AddInt (&sl, pl->ob->face->number);
1232
1233 SockList_AddChar (&sl, (char) strlen (pl->ob->name));
1234 strcpy ((char *) sl.buf + sl.len, pl->ob->name);
1235 sl.len += strlen (pl->ob->name);
1236 1211
1237 Send_With_Handling (&pl->socket, &sl); 1212 Send_With_Handling (&pl->socket, &sl);
1238 free (sl.buf); 1213 sl.free ();
1239 SET_FLAG (pl->ob, FLAG_CLIENT_SENT); 1214 SET_FLAG (pl->ob, FLAG_CLIENT_SENT);
1240} 1215}
1241 1216
1242/** 1217/**
1243 * Need to send an animation sequence to the client. 1218 * Need to send an animation sequence to the client.
1247 * when the player logs in and picks stuff up. 1222 * when the player logs in and picks stuff up.
1248 */ 1223 */
1249void 1224void
1250esrv_send_animation (NewSocket * ns, short anim_num) 1225esrv_send_animation (NewSocket * ns, short anim_num)
1251{ 1226{
1252 SockList sl;
1253 int i; 1227 int i;
1254 1228
1255 /* Do some checking on the anim_num we got. Note that the animations 1229 /* Do some checking on the anim_num we got. Note that the animations
1256 * are added in contigous order, so if the number is in the valid 1230 * are added in contigous order, so if the number is in the valid
1257 * range, it must be a valid animation. 1231 * range, it must be a valid animation.
1260 { 1234 {
1261 LOG (llevError, "esrv_send_anim (%d) out of bounds??\n", anim_num); 1235 LOG (llevError, "esrv_send_anim (%d) out of bounds??\n", anim_num);
1262 return; 1236 return;
1263 } 1237 }
1264 1238
1265 sl.buf = (unsigned char *) malloc (MAXSOCKBUF); 1239 SockList sl (MAXSOCKBUF);
1266 strcpy ((char *) sl.buf, "anim "); 1240
1267 sl.len = 5; 1241 sl << "anim "
1268 SockList_AddShort (&sl, anim_num); 1242 << uint16 (anim_num)
1269 SockList_AddShort (&sl, 0); /* flags - not used right now */ 1243 << uint16 (0); /* flags - not used right now */
1244
1270 /* Build up the list of faces. Also, send any information (ie, the 1245 /* Build up the list of faces. Also, send any information (ie, the
1271 * the face itself) down to the client. 1246 * the face itself) down to the client.
1272 */ 1247 */
1273 for (i = 0; i < animations[anim_num].num_animations; i++) 1248 for (i = 0; i < animations[anim_num].num_animations; i++)
1274 { 1249 {
1275 if (!(ns->faces_sent[animations[anim_num].faces[i]] & NS_FACESENT_FACE)) 1250 if (!(ns->faces_sent[animations[anim_num].faces[i]] & NS_FACESENT_FACE))
1276 esrv_send_face (ns, animations[anim_num].faces[i], 0); 1251 esrv_send_face (ns, animations[anim_num].faces[i], 0);
1277 SockList_AddShort (&sl, animations[anim_num].faces[i]); /* flags - not used right now */ 1252 sl << uint16 (animations[anim_num].faces[i]); /* flags - not used right now */
1278 } 1253 }
1254
1279 Send_With_Handling (ns, &sl); 1255 Send_With_Handling (ns, &sl);
1280 free (sl.buf); 1256 sl.free ();
1257
1281 ns->anims_sent[anim_num] = 1; 1258 ns->anims_sent[anim_num] = 1;
1282} 1259}
1283 1260
1284 1261
1285/****************************************************************************** 1262/******************************************************************************
1299 { 1276 {
1300 LOG (llevError, "Too many faces in map cell %d %d\n", x, y); 1277 LOG (llevError, "Too many faces in map cell %d %d\n", x, y);
1301 return; 1278 return;
1302 abort (); 1279 abort ();
1303 } 1280 }
1281
1304 newmap->cells[x][y].faces[newmap->cells[x][y].count] = face_num; 1282 newmap->cells[x][y].faces[newmap->cells[x][y].count] = face_num;
1305 newmap->cells[x][y].count++; 1283 newmap->cells[x][y].count++;
1284
1306 if (!(ns->faces_sent[face_num] & NS_FACESENT_FACE)) 1285 if (!(ns->faces_sent[face_num] & NS_FACESENT_FACE))
1307 esrv_send_face (ns, face_num, 0); 1286 esrv_send_face (ns, face_num, 0);
1308} 1287}
1309 1288
1310struct LayerCell 1289struct LayerCell
1417static void 1396static void
1418esrv_map_doneredraw (NewSocket * ns, struct Map *newmap) 1397esrv_map_doneredraw (NewSocket * ns, struct Map *newmap)
1419{ 1398{
1420 static long frames, bytes, tbytes, tframes; 1399 static long frames, bytes, tbytes, tframes;
1421 char *cur; 1400 char *cur;
1422 SockList sl;
1423 1401
1424 1402 SockList sl (MAXSOCKBUF);
1425 sl.buf = (unsigned char *) malloc (MAXSOCKBUF); 1403 sl << "map ";
1426 strcpy ((char *) sl.buf, "map ");
1427 sl.len = strlen ((char *) sl.buf);
1428 1404
1429 cur = (char *) compactlayer (ns, (unsigned char *) sl.buf + sl.len, MAP_LAYERS, newmap); 1405 cur = (char *) compactlayer (ns, (unsigned char *) sl.buf + sl.len, MAP_LAYERS, newmap);
1430 sl.len = cur - (char *) sl.buf; 1406 sl.len = cur - (char *) sl.buf;
1431 1407
1432/* LOG(llevDebug, "Sending map command.\n");*/ 1408/* LOG(llevDebug, "Sending map command.\n");*/
1444 memcpy (&ns->lastmap, newmap, sizeof (struct Map)); 1420 memcpy (&ns->lastmap, newmap, sizeof (struct Map));
1445 Send_With_Handling (ns, &sl); 1421 Send_With_Handling (ns, &sl);
1446 ns->sent_scroll = 0; 1422 ns->sent_scroll = 0;
1447 } 1423 }
1448 1424
1449 free (sl.buf); 1425 sl.free ();
1450} 1426}
1451 1427
1452 1428
1453/** Clears a map cell */ 1429/** Clears a map cell */
1454static void 1430static void
1496 * needs to get sent - if so, it adds the data, sending the head 1472 * needs to get sent - if so, it adds the data, sending the head
1497 * if needed, and returning 1. If this no data needs to get 1473 * if needed, and returning 1. If this no data needs to get
1498 * sent, it returns zero. 1474 * sent, it returns zero.
1499 */ 1475 */
1500static int 1476static int
1501check_head (SockList & sl, NewSocket & ns, int ax, int ay, int layer) 1477check_head (SockList &sl, NewSocket &ns, int ax, int ay, int layer)
1502{ 1478{
1503 short face_num; 1479 short face_num;
1504 1480
1505 if (heads[(ay * MAX_HEAD_POS + ax) * MAX_LAYERS + layer]) 1481 if (heads[(ay * MAX_HEAD_POS + ax) * MAX_LAYERS + layer])
1506 face_num = heads[(ay * MAX_HEAD_POS + ax) * MAX_LAYERS + layer]->face->number; 1482 face_num = heads[(ay * MAX_HEAD_POS + ax) * MAX_LAYERS + layer]->face->number;
1507 else 1483 else
1508 face_num = 0; 1484 face_num = 0;
1509 1485
1510 if (face_num != ns.lastmap.cells[ax][ay].faces[layer]) 1486 if (face_num != ns.lastmap.cells[ax][ay].faces[layer])
1511 { 1487 {
1512 SockList_AddShort (&sl, face_num); 1488 sl << uint16 (face_num);
1513 if (face_num && !(ns.faces_sent[face_num] & NS_FACESENT_FACE)) 1489 if (face_num && !(ns.faces_sent[face_num] & NS_FACESENT_FACE))
1514 esrv_send_face (&ns, face_num, 0); 1490 esrv_send_face (&ns, face_num, 0);
1491
1515 heads[(ay * MAX_HEAD_POS + ax) * MAX_LAYERS + layer] = NULL; 1492 heads[(ay * MAX_HEAD_POS + ax) * MAX_LAYERS + layer] = NULL;
1516 ns.lastmap.cells[ax][ay].faces[layer] = face_num; 1493 ns.lastmap.cells[ax][ay].faces[layer] = face_num;
1517 return 1; 1494 return 1;
1518 } 1495 }
1519 1496
1539 * the case, it seems to make more sense to have these layer values 1516 * the case, it seems to make more sense to have these layer values
1540 * actually match. 1517 * actually match.
1541 */ 1518 */
1542 1519
1543static int 1520static int
1544update_space (SockList * sl, NewSocket * ns, maptile *mp, int mx, int my, int sx, int sy, int layer) 1521update_space (SockList &sl, NewSocket &ns, maptile *mp, int mx, int my, int sx, int sy, int layer)
1545{ 1522{
1546 object *ob, *head; 1523 object *ob, *head;
1547 uint16 face_num; 1524 uint16 face_num;
1548 int bx, by, i; 1525 int bx, by, i;
1549 1526
1550 /* If there is a multipart object stored away, treat that as more important. 1527 /* If there is a multipart object stored away, treat that as more important.
1551 * If not, then do the normal processing. 1528 * If not, then do the normal processing.
1552 */ 1529 */
1553
1554 head = heads[(sy * MAX_HEAD_POS + sx) * MAX_LAYERS + layer]; 1530 head = heads[(sy * MAX_HEAD_POS + sx) * MAX_LAYERS + layer];
1555 1531
1556 /* Check to see if this head is part of the set of objects 1532 /* Check to see if this head is part of the set of objects
1557 * we would normally send for this space. If so, then 1533 * we would normally send for this space. If so, then
1558 * don't use the head value. We need to do the check 1534 * don't use the head value. We need to do the check
1723 1699
1724 /* Another hack - because of heads and whatnot, this face may match one 1700 /* Another hack - because of heads and whatnot, this face may match one
1725 * we already sent for a lower layer. In that case, don't send 1701 * we already sent for a lower layer. In that case, don't send
1726 * this one. 1702 * this one.
1727 */ 1703 */
1728 if (face_num && layer + 1 < MAP_LAYERS && ns->lastmap.cells[sx][sy].faces[layer + 1] == face_num) 1704 if (face_num && layer + 1 < MAP_LAYERS && ns.lastmap.cells[sx][sy].faces[layer + 1] == face_num)
1729 {
1730 face_num = 0; 1705 face_num = 0;
1731 }
1732 1706
1733 /* We've gotten what face we want to use for the object. Now see if 1707 /* We've gotten what face we want to use for the object. Now see if
1734 * if it has changed since we last sent it to the client. 1708 * if it has changed since we last sent it to the client.
1735 */ 1709 */
1736 if (ns->lastmap.cells[sx][sy].faces[layer] != face_num) 1710 if (ns.lastmap.cells[sx][sy].faces[layer] != face_num)
1737 { 1711 {
1738 ns->lastmap.cells[sx][sy].faces[layer] = face_num; 1712 ns.lastmap.cells[sx][sy].faces[layer] = face_num;
1739 if (!(ns->faces_sent[face_num] & NS_FACESENT_FACE)) 1713 if (!(ns.faces_sent[face_num] & NS_FACESENT_FACE))
1740 esrv_send_face (ns, face_num, 0); 1714 esrv_send_face (&ns, face_num, 0);
1741 SockList_AddShort (sl, face_num); 1715
1716 sl << uint16 (face_num);
1742 return 1; 1717 return 1;
1743 } 1718 }
1719
1744 /* Nothing changed */ 1720 /* Nothing changed */
1745 return 0; 1721 return 0;
1746} 1722}
1747 1723
1748/** 1724/**
1763 * top layer (this matches what the GET_MAP_FACE and GET_MAP_FACE_OBJ 1739 * top layer (this matches what the GET_MAP_FACE and GET_MAP_FACE_OBJ
1764 * take. 1740 * take.
1765 */ 1741 */
1766 1742
1767static inline int 1743static inline int
1768update_smooth (SockList * sl, NewSocket * ns, maptile *mp, int mx, int my, int sx, int sy, int layer) 1744update_smooth (SockList &sl, NewSocket &ns, maptile *mp, int mx, int my, int sx, int sy, int layer)
1769{ 1745{
1770 object *ob; 1746 object *ob;
1771 int smoothlevel; /* old face_num; */ 1747 int smoothlevel; /* old face_num; */
1772 1748
1773 ob = GET_MAP_FACE_OBJ (mp, mx, my, layer); 1749 ob = GET_MAP_FACE_OBJ (mp, mx, my, layer);
1778 if (!ob || ob->face == blank_face || MAP_NOSMOOTH (mp)) 1754 if (!ob || ob->face == blank_face || MAP_NOSMOOTH (mp))
1779 smoothlevel = 0; 1755 smoothlevel = 0;
1780 else 1756 else
1781 { 1757 {
1782 smoothlevel = ob->smoothlevel; 1758 smoothlevel = ob->smoothlevel;
1783 if (smoothlevel && !(ns->faces_sent[ob->face->number] & NS_FACESENT_SMOOTH)) 1759 if (smoothlevel && !(ns.faces_sent[ob->face->number] & NS_FACESENT_SMOOTH))
1784 SendSmooth (ns, ob->face->number); 1760 SendSmooth (&ns, ob->face->number);
1785 } /* else not already head object or blank face */ 1761 } /* else not already head object or blank face */
1786 1762
1787 /* We've gotten what face we want to use for the object. Now see if 1763 /* We've gotten what face we want to use for the object. Now see if
1788 * if it has changed since we last sent it to the client. 1764 * if it has changed since we last sent it to the client.
1789 */ 1765 */
1790 if (smoothlevel > 255) 1766 if (smoothlevel > 255)
1791 smoothlevel = 255; 1767 smoothlevel = 255;
1792 else if (smoothlevel < 0) 1768 else if (smoothlevel < 0)
1793 smoothlevel = 0; 1769 smoothlevel = 0;
1770
1794 if (ns->lastmap.cells[sx][sy].smooth[layer] != smoothlevel) 1771 if (ns.lastmap.cells[sx][sy].smooth[layer] != smoothlevel)
1795 { 1772 {
1796 ns->lastmap.cells[sx][sy].smooth[layer] = smoothlevel; 1773 ns.lastmap.cells[sx][sy].smooth[layer] = smoothlevel;
1797 SockList_AddChar (sl, (uint8) (smoothlevel & 0xFF)); 1774 sl << uint8 (smoothlevel);
1798 return 1; 1775 return 1;
1799 } 1776 }
1777
1800 /* Nothing changed */ 1778 /* Nothing changed */
1801 return 0; 1779 return 0;
1802} 1780}
1803 1781
1804/** 1782/**
1845draw_client_map1 (object *pl) 1823draw_client_map1 (object *pl)
1846{ 1824{
1847 int x, y, ax, ay, d, startlen, max_x, max_y, oldlen; 1825 int x, y, ax, ay, d, startlen, max_x, max_y, oldlen;
1848 sint16 nx, ny; 1826 sint16 nx, ny;
1849 int estartlen, eoldlen; 1827 int estartlen, eoldlen;
1850 SockList sl;
1851 SockList esl; /*For extended Map info */
1852 uint16 mask, emask; 1828 uint16 mask, emask;
1853 uint8 eentrysize; 1829 uint8 eentrysize;
1854 uint16 ewhatstart, ewhatflag; 1830 uint16 ewhatstart, ewhatflag;
1855 uint8 extendedinfos; 1831 uint8 extendedinfos;
1856 maptile *m; 1832 maptile *m;
1857 1833
1858 NewSocket & socket = pl->contr->socket; 1834 NewSocket &socket = pl->contr->socket;
1859 1835
1860 check_map_change (pl->contr); 1836 check_map_change (pl->contr);
1861 1837
1862 sl.buf = (unsigned char *) malloc (MAXSOCKBUF); 1838 SockList sl (MAXSOCKBUF);
1863 if (socket.mapmode == Map1Cmd) 1839 SockList esl (MAXSOCKBUF);
1864 strcpy ((char *) sl.buf, "map1 "); 1840
1865 else 1841 sl << (socket.mapmode == Map1Cmd ? "map1 " : "map1a ");
1866 strcpy ((char *) sl.buf, "map1a ");
1867 sl.len = strlen ((char *) sl.buf);
1868 startlen = sl.len; 1842 startlen = sl.len;
1869 1843
1870 /*Extendedmapinfo structure initialisation */ 1844 /*Extendedmapinfo structure initialisation */
1871 if (socket.ext_mapinfos) 1845 if (socket.ext_mapinfos)
1872 { 1846 {
1873 esl.buf = (unsigned char *) malloc (MAXSOCKBUF);
1874 strcpy ((char *) esl.buf, "mapextended ");
1875 esl.len = strlen ((char *) esl.buf);
1876 extendedinfos = EMI_NOREDRAW; 1847 extendedinfos = EMI_NOREDRAW;
1877 1848
1878 if (socket.EMI_smooth) 1849 if (socket.EMI_smooth)
1879 extendedinfos |= EMI_SMOOTH; 1850 extendedinfos |= EMI_SMOOTH;
1880 1851
1881 ewhatstart = esl.len; 1852 ewhatstart = esl.len;
1882 ewhatflag = extendedinfos; /*The EMI_NOREDRAW bit 1853 ewhatflag = extendedinfos; /*The EMI_NOREDRAW bit
1883 could need to be taken away */ 1854 could need to be taken away */
1884 SockList_AddChar (&esl, extendedinfos);
1885 eentrysize = getExtendedMapInfoSize (&socket); 1855 eentrysize = getExtendedMapInfoSize (&socket);
1886 SockList_AddChar (&esl, eentrysize); 1856 esl << "mapextended "
1857 << uint8 (extendedinfos)
1858 << uint8 (eentrysize);
1887 estartlen = esl.len; 1859 estartlen = esl.len;
1888 } 1860 }
1889 else 1861 else
1890 { 1862 {
1891 /* suppress compiler warnings */ 1863 /* suppress compiler warnings */
1932 { 1904 {
1933 int i, got_one; 1905 int i, got_one;
1934 1906
1935 oldlen = sl.len; 1907 oldlen = sl.len;
1936 1908
1937 SockList_AddShort (&sl, mask); 1909 sl << uint16 (mask);
1938 1910
1939 if (check_head (sl, socket, ax, ay, 2)) 1911 if (check_head (sl, socket, ax, ay, 2)) mask |= 0x4;
1940 mask |= 0x4;
1941 if (check_head (sl, socket, ax, ay, 1)) 1912 if (check_head (sl, socket, ax, ay, 1)) mask |= 0x2;
1942 mask |= 0x2;
1943 if (check_head (sl, socket, ax, ay, 0)) 1913 if (check_head (sl, socket, ax, ay, 0)) mask |= 0x1;
1944 mask |= 0x1;
1945 1914
1946 /* If all we are doing is sending 0 (blank) faces, we don't 1915 /* If all we are doing is sending 0 (blank) faces, we don't
1947 * actually need to send that - just the coordinates 1916 * actually need to send that - just the coordinates
1948 * with no faces tells the client to blank out the 1917 * with no faces tells the client to blank out the
1949 * space. 1918 * space.
1986 * if this hasn't already been done. If the space is out 1955 * if this hasn't already been done. If the space is out
1987 * of the map, it shouldn't have a head 1956 * of the map, it shouldn't have a head
1988 */ 1957 */
1989 if (lastcell.count != -1) 1958 if (lastcell.count != -1)
1990 { 1959 {
1991 SockList_AddShort (&sl, mask); 1960 sl << uint16 (mask);
1992 map_clearcell (&lastcell, 0, 0, 0, -1); 1961 map_clearcell (&lastcell, 0, 0, 0, -1);
1993 } 1962 }
1994 } 1963 }
1995 else if (d > 3) 1964 else if (d > 3)
1996 { 1965 {
2000 * reason. Still may need to send the head for this space. 1969 * reason. Still may need to send the head for this space.
2001 */ 1970 */
2002 1971
2003 oldlen = sl.len; 1972 oldlen = sl.len;
2004 1973
2005 SockList_AddShort (&sl, mask); 1974 sl << uint16 (mask);
2006 1975
2007 if (lastcell.count != -1) 1976 if (lastcell.count != -1)
2008 need_send = 1; 1977 need_send = 1;
2009 1978
2010 count = -1; 1979 count = -1;
2011 1980
2012 if (socket.mapmode == Map1aCmd && have_head (ax, ay)) 1981 if (socket.mapmode == Map1aCmd && have_head (ax, ay))
2013 { 1982 {
2014 /* Now check to see if any heads need to be sent */ 1983 /* Now check to see if any heads need to be sent */
2015 1984
2016 if (check_head (sl, socket, ax, ay, 2)) 1985 if (check_head (sl, socket, ax, ay, 2)) mask |= 0x4;
2017 mask |= 0x4;
2018 if (check_head (sl, socket, ax, ay, 1)) 1986 if (check_head (sl, socket, ax, ay, 1)) mask |= 0x2;
2019 mask |= 0x2;
2020 if (check_head (sl, socket, ax, ay, 0)) 1987 if (check_head (sl, socket, ax, ay, 0)) mask |= 0x1;
2021 mask |= 0x1;
2022 1988
2023 lastcell.count = count; 1989 lastcell.count = count;
2024 } 1990 }
2025 else 1991 else
2026 { 1992 {
2055 */ 2021 */
2056 oldlen = sl.len; 2022 oldlen = sl.len;
2057 mask = (ax & 0x3f) << 10 | (ay & 0x3f) << 4; 2023 mask = (ax & 0x3f) << 10 | (ay & 0x3f) << 4;
2058 eoldlen = esl.len; 2024 eoldlen = esl.len;
2059 emask = (ax & 0x3f) << 10 | (ay & 0x3f) << 4; 2025 emask = (ax & 0x3f) << 10 | (ay & 0x3f) << 4;
2060 SockList_AddShort (&sl, mask); 2026 sl << uint16 (mask);
2061 2027
2062 if (socket.ext_mapinfos) 2028 if (socket.ext_mapinfos)
2063 SockList_AddShort (&esl, emask); 2029 esl << uint16 (emask);
2064 2030
2065 unsigned char dummy; 2031 unsigned char dummy;
2066 unsigned char *last_ext = &dummy; 2032 unsigned char *last_ext = &dummy;
2067 2033
2068 /* Darkness changed */ 2034 /* Darkness changed */
2072 2038
2073 if (socket.extmap) 2039 if (socket.extmap)
2074 { 2040 {
2075 *last_ext |= 0x80; 2041 *last_ext |= 0x80;
2076 last_ext = sl.buf + sl.len; 2042 last_ext = sl.buf + sl.len;
2077 SockList_AddChar (&sl, d); 2043 sl << uint8 (d);
2078 } 2044 }
2079 else 2045 else
2080 SockList_AddChar (&sl, 255 - 64 * d); 2046 sl << uint8 (255 - 64 * d);
2081 } 2047 }
2082 2048
2083 lastcell.count = d; 2049 lastcell.count = d;
2084 2050
2085 if (socket.extmap) 2051 if (socket.extmap)
2152 sl << uint8 (8) << uint8 (flags); 2118 sl << uint8 (8) << uint8 (flags);
2153 } 2119 }
2154 } 2120 }
2155 2121
2156 /* Floor face */ 2122 /* Floor face */
2157 if (update_space (&sl, &socket, m, nx, ny, ax, ay, 2)) 2123 if (update_space (sl, socket, m, nx, ny, ax, ay, 2))
2158 mask |= 0x4; 2124 mask |= 0x4;
2159 2125
2160 if (socket.EMI_smooth) 2126 if (socket.EMI_smooth)
2161 if (update_smooth (&esl, &socket, m, nx, ny, ax, ay, 2)) 2127 if (update_smooth (esl, socket, m, nx, ny, ax, ay, 2))
2162 emask |= 0x4; 2128 emask |= 0x4;
2163 2129
2164 /* Middle face */ 2130 /* Middle face */
2165 if (update_space (&sl, &socket, m, nx, ny, ax, ay, 1)) 2131 if (update_space (sl, socket, m, nx, ny, ax, ay, 1))
2166 mask |= 0x2; 2132 mask |= 0x2;
2167 2133
2168 if (socket.EMI_smooth) 2134 if (socket.EMI_smooth)
2169 if (update_smooth (&esl, &socket, m, nx, ny, ax, ay, 1)) 2135 if (update_smooth (esl, socket, m, nx, ny, ax, ay, 1))
2170 emask |= 0x2; 2136 emask |= 0x2;
2171 2137
2172 if (nx == pl->x && ny == pl->y && pl->invisible & (pl->invisible < 50 ? 4 : 1)) 2138 if (nx == pl->x && ny == pl->y && pl->invisible & (pl->invisible < 50 ? 4 : 1))
2173 { 2139 {
2174 if (lastcell.faces[0] != pl->face->number) 2140 if (lastcell.faces[0] != pl->face->number)
2183 } 2149 }
2184 } 2150 }
2185 else 2151 else
2186 { 2152 {
2187 /* Top face */ 2153 /* Top face */
2188 if (update_space (&sl, &socket, m, nx, ny, ax, ay, 0)) 2154 if (update_space (sl, socket, m, nx, ny, ax, ay, 0))
2189 mask |= 0x1; 2155 mask |= 0x1;
2190 2156
2191 if (socket.EMI_smooth) 2157 if (socket.EMI_smooth)
2192 if (update_smooth (&esl, &socket, m, nx, ny, ax, ay, 0)) 2158 if (update_smooth (esl, socket, m, nx, ny, ax, ay, 0))
2193 emask |= 0x1; 2159 emask |= 0x1;
2194 } 2160 }
2195 2161
2196 /* Check to see if we are in fact sending anything for this 2162 /* Check to see if we are in fact sending anything for this
2197 * space by checking the mask. If so, update the mask. 2163 * space by checking the mask. If so, update the mask.
2223 esl.buf[ewhatstart + 1] = ewhatflag & 0xff; 2189 esl.buf[ewhatstart + 1] = ewhatflag & 0xff;
2224 } 2190 }
2225 2191
2226 if (esl.len > estartlen) 2192 if (esl.len > estartlen)
2227 Send_With_Handling (&socket, &esl); 2193 Send_With_Handling (&socket, &esl);
2228
2229 free (esl.buf);
2230 } 2194 }
2231 2195
2232 if (sl.len > startlen || socket.sent_scroll) 2196 if (sl.len > startlen || socket.sent_scroll)
2233 { 2197 {
2234 Send_With_Handling (&socket, &sl); 2198 Send_With_Handling (&socket, &sl);
2235 socket.sent_scroll = 0; 2199 socket.sent_scroll = 0;
2236 } 2200 }
2237 2201
2238 free (sl.buf); 2202 sl.free ();
2203 esl.free ();
2239} 2204}
2240 2205
2241/** 2206/**
2242 * Draws client map. 2207 * Draws client map.
2243 */ 2208 */
2357 esrv_map_setbelow (&pl->contr->socket, ax, ay, floor->number, &newmap); 2322 esrv_map_setbelow (&pl->contr->socket, ax, ay, floor->number, &newmap);
2358 } 2323 }
2359 } /* Is a valid space */ 2324 } /* Is a valid space */
2360 } 2325 }
2361 } 2326 }
2327
2362 esrv_map_doneredraw (&pl->contr->socket, &newmap); 2328 esrv_map_doneredraw (&pl->contr->socket, &newmap);
2363 2329
2364 check_map_change (pl->contr); 2330 check_map_change (pl->contr);
2365} 2331}
2366 2332
2383/** 2349/**
2384 * This sends the skill number to name mapping. We ignore 2350 * This sends the skill number to name mapping. We ignore
2385 * the params - we always send the same info no matter what. 2351 * the params - we always send the same info no matter what.
2386 */ 2352 */
2387void 2353void
2388send_skill_info (NewSocket * ns, char *params) 2354send_skill_info (NewSocket *ns, char *params)
2389{ 2355{
2390 SockList sl; 2356 SockList sl (MAXSOCKBUF);
2391 int i; 2357 sl << "replyinfo skill_info\n";
2392 2358
2393 sl.buf = (unsigned char *) malloc (MAXSOCKBUF);
2394 strcpy ((char *) sl.buf, "replyinfo skill_info\n");
2395 for (i = 1; i < NUM_SKILLS; i++) 2359 for (int i = 1; i < NUM_SKILLS; i++)
2396 { 2360 sl.printf ("%d:%s\n", i + CS_STAT_SKILLINFO, &skill_names[i]);
2397 sprintf ((char *) sl.buf + strlen ((char *) sl.buf), "%d:%s\n", i + CS_STAT_SKILLINFO, &skill_names[i]); 2361
2398 }
2399 sl.len = strlen ((char *) sl.buf);
2400 if (sl.len >= MAXSOCKBUF) 2362 if (sl.len >= MAXSOCKBUF)
2401 { 2363 {
2402 LOG (llevError, "Buffer overflow in send_skill_info!\n"); 2364 LOG (llevError, "Buffer overflow in send_skill_info!\n");
2403 fatal (0); 2365 fatal (0);
2404 } 2366 }
2367
2405 Send_With_Handling (ns, &sl); 2368 Send_With_Handling (ns, &sl);
2406 free (sl.buf); 2369 sl.free ();
2407} 2370}
2408 2371
2409/** 2372/**
2410 * This sends the spell path to name mapping. We ignore 2373 * This sends the spell path to name mapping. We ignore
2411 * the params - we always send the same info no matter what. 2374 * the params - we always send the same info no matter what.
2412 */ 2375 */
2413void 2376void
2414send_spell_paths (NewSocket * ns, char *params) 2377send_spell_paths (NewSocket * ns, char *params)
2415{ 2378{
2416 SockList sl; 2379 SockList sl (MAXSOCKBUF);
2417 int i;
2418 2380
2419 sl.buf = (unsigned char *) malloc (MAXSOCKBUF); 2381 sl << "replyinfo spell_paths\n";
2420 strcpy ((char *) sl.buf, "replyinfo spell_paths\n"); 2382
2421 for (i = 0; i < NRSPELLPATHS; i++) 2383 for (int i = 0; i < NRSPELLPATHS; i++)
2422 sprintf ((char *) sl.buf + strlen ((char *) sl.buf), "%d:%s\n", 1 << i, spellpathnames[i]); 2384 sl.printf ("%d:%s\n", 1 << i, spellpathnames[i]);
2423 sl.len = strlen ((char *) sl.buf); 2385
2424 if (sl.len >= MAXSOCKBUF) 2386 if (sl.len >= MAXSOCKBUF)
2425 { 2387 {
2426 LOG (llevError, "Buffer overflow in send_spell_paths!\n"); 2388 LOG (llevError, "Buffer overflow in send_spell_paths!\n");
2427 fatal (0); 2389 fatal (0);
2428 } 2390 }
2391
2429 Send_With_Handling (ns, &sl); 2392 Send_With_Handling (ns, &sl);
2430 free (sl.buf); 2393 sl.free ();
2431} 2394}
2432 2395
2433/** 2396/**
2434 * This looks for any spells the player may have that have changed their stats. 2397 * This looks for any spells the player may have that have changed their stats.
2435 * it then sends an updspell packet for each spell that has changed in this way 2398 * it then sends an updspell packet for each spell that has changed in this way
2436 */ 2399 */
2437void 2400void
2438esrv_update_spells (player *pl) 2401esrv_update_spells (player *pl)
2439{ 2402{
2440 SockList sl;
2441 int flags = 0;
2442 object *spell;
2443
2444 if (!pl->socket.monitor_spells) 2403 if (!pl->socket.monitor_spells)
2445 return; 2404 return;
2405
2446 for (spell = pl->ob->inv; spell != NULL; spell = spell->below) 2406 for (object *spell = pl->ob->inv; spell; spell = spell->below)
2447 { 2407 {
2448 if (spell->type == SPELL) 2408 if (spell->type == SPELL)
2449 { 2409 {
2410 int flags = 0;
2411
2450 /* check if we need to update it */ 2412 /* check if we need to update it */
2451 if (spell->last_sp != SP_level_spellpoint_cost (pl->ob, spell, SPELL_MANA)) 2413 if (spell->last_sp != SP_level_spellpoint_cost (pl->ob, spell, SPELL_MANA))
2452 { 2414 {
2453 spell->last_sp = SP_level_spellpoint_cost (pl->ob, spell, SPELL_MANA); 2415 spell->last_sp = SP_level_spellpoint_cost (pl->ob, spell, SPELL_MANA);
2454 flags |= UPD_SP_MANA; 2416 flags |= UPD_SP_MANA;
2455 } 2417 }
2418
2456 if (spell->last_grace != SP_level_spellpoint_cost (pl->ob, spell, SPELL_GRACE)) 2419 if (spell->last_grace != SP_level_spellpoint_cost (pl->ob, spell, SPELL_GRACE))
2457 { 2420 {
2458 spell->last_grace = SP_level_spellpoint_cost (pl->ob, spell, SPELL_GRACE); 2421 spell->last_grace = SP_level_spellpoint_cost (pl->ob, spell, SPELL_GRACE);
2459 flags |= UPD_SP_GRACE; 2422 flags |= UPD_SP_GRACE;
2460 } 2423 }
2424
2461 if (spell->last_eat != spell->stats.dam + SP_level_dam_adjust (pl->ob, spell)) 2425 if (spell->last_eat != spell->stats.dam + SP_level_dam_adjust (pl->ob, spell))
2462 { 2426 {
2463 spell->last_eat = spell->stats.dam + SP_level_dam_adjust (pl->ob, spell); 2427 spell->last_eat = spell->stats.dam + SP_level_dam_adjust (pl->ob, spell);
2464 flags |= UPD_SP_DAMAGE; 2428 flags |= UPD_SP_DAMAGE;
2465 } 2429 }
2430
2466 if (flags != 0) 2431 if (flags)
2467 { 2432 {
2468 sl.buf = (unsigned char *) malloc (MAXSOCKBUF); 2433 SockList sl (MAXSOCKBUF);
2469 strcpy ((char *) sl.buf, "updspell ");
2470 sl.len = strlen ((char *) sl.buf);
2471 2434
2435 sl << "updspell "
2436 << uint8 (flags)
2472 sl << uint8 (flags) << uint32 (spell->count); 2437 << uint32 (spell->count);
2473 2438
2474 if (flags & UPD_SP_MANA ) sl << uint16 (spell->last_sp); 2439 if (flags & UPD_SP_MANA ) sl << uint16 (spell->last_sp);
2475 if (flags & UPD_SP_GRACE ) sl << uint16 (spell->last_grace); 2440 if (flags & UPD_SP_GRACE ) sl << uint16 (spell->last_grace);
2476 if (flags & UPD_SP_DAMAGE) sl << uint16 (spell->last_eat); 2441 if (flags & UPD_SP_DAMAGE) sl << uint16 (spell->last_eat);
2477 2442
2478 flags = 0;
2479 Send_With_Handling (&pl->socket, &sl); 2443 Send_With_Handling (&pl->socket, &sl);
2480 free (sl.buf); 2444 sl.free ();
2481 } 2445 }
2482 } 2446 }
2483 } 2447 }
2484} 2448}
2485 2449
2486void 2450void
2487esrv_remove_spell (player *pl, object *spell) 2451esrv_remove_spell (player *pl, object *spell)
2488{ 2452{
2489 SockList sl;
2490
2491 if (!pl->socket.monitor_spells) 2453 if (!pl->socket.monitor_spells)
2492 return; 2454 return;
2455
2493 if (!pl || !spell || spell->env != pl->ob) 2456 if (!pl || !spell || spell->env != pl->ob)
2494 { 2457 {
2495 LOG (llevError, "Invalid call to esrv_remove_spell"); 2458 LOG (llevError, "Invalid call to esrv_remove_spell");
2496 return; 2459 return;
2497 } 2460 }
2498 sl.buf = (unsigned char *) malloc (MAXSOCKBUF);
2499 strcpy ((char *) sl.buf, "delspell ");
2500 sl.len = strlen ((char *) sl.buf);
2501 2461
2462 SockList sl (MAXSOCKBUF);
2463
2464 sl << "delspell "
2502 sl << uint32 (spell->count); 2465 << uint32 (spell->count);
2503 2466
2504 Send_With_Handling (&pl->socket, &sl); 2467 Send_With_Handling (&pl->socket, &sl);
2505 free (sl.buf); 2468 sl.free ();
2506} 2469}
2507 2470
2508/* appends the spell *spell to the Socklist we will send the data to. */ 2471/* appends the spell *spell to the Socklist we will send the data to. */
2509static void 2472static void
2510append_spell (player *pl, SockList &sl, object *spell) 2473append_spell (player *pl, SockList &sl, object *spell)
2552 * all spells in the player's inventory. 2515 * all spells in the player's inventory.
2553 */ 2516 */
2554void 2517void
2555esrv_add_spells (player *pl, object *spell) 2518esrv_add_spells (player *pl, object *spell)
2556{ 2519{
2557 SockList sl;
2558
2559 if (!pl) 2520 if (!pl)
2560 { 2521 {
2561 LOG (llevError, "esrv_add_spells, tried to add a spell to a NULL player"); 2522 LOG (llevError, "esrv_add_spells, tried to add a spell to a NULL player");
2562 return; 2523 return;
2563 } 2524 }
2525
2564 if (!pl->socket.monitor_spells) 2526 if (!pl->socket.monitor_spells)
2565 return; 2527 return;
2566 sl.buf = (unsigned char *) malloc (MAXSOCKBUF); 2528
2567 strcpy ((char *) sl.buf, "addspell "); 2529 SockList sl (MAXSOCKBUF);
2568 sl.len = strlen ((char *) sl.buf); 2530
2531 sl << "addspell ";
2532
2569 if (!spell) 2533 if (!spell)
2570 { 2534 {
2571 for (spell = pl->ob->inv; spell != NULL; spell = spell->below) 2535 for (spell = pl->ob->inv; spell != NULL; spell = spell->below)
2572 { 2536 {
2573 /* were we to simply keep appending data here, we could exceed 2537 /* were we to simply keep appending data here, we could exceed
2602 return; 2566 return;
2603 } 2567 }
2604 else 2568 else
2605 append_spell (pl, sl, spell); 2569 append_spell (pl, sl, spell);
2606 2570
2607 if (sl.len >= MAXSOCKBUF) 2571 if (sl.length () >= MAXSOCKBUF)
2608 { 2572 {
2609 LOG (llevError, "Buffer overflow in esrv_add_spells!\n"); 2573 LOG (llevError, "Buffer overflow in esrv_add_spells!\n");
2610 fatal (0); 2574 fatal (0);
2611 } 2575 }
2612 2576
2613 /* finally, we can send the packet */ 2577 /* finally, we can send the packet */
2614 Send_With_Handling (&pl->socket, &sl); 2578 Send_With_Handling (&pl->socket, &sl);
2615 free (sl.buf); 2579
2580 sl.free ();
2616} 2581}
2582

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines