home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #31 / NN_1992_31.iso / spool / alt / games / gb / 1380 < prev    next >
Encoding:
Internet Message Format  |  1992-12-25  |  7.7 KB
  1. Path: sparky!uunet!cs.utexas.edu!gateway
  2. From: deragon@JETHRO.CS.NYU.EDU (John Deragon)
  3. Newsgroups: alt.games.gb
  4. Subject: patches for bid()
  5. Date: 25 Dec 1992 03:25:21 -0600
  6. Organization: UTexas Mail-to-News Gateway
  7. Lines: 278
  8. Sender: daemon@cs.utexas.edu
  9. Message-ID: <9212250923.AA22881@JETHRO.CS.NYU.EDU>
  10. NNTP-Posting-Host: cs.utexas.edu
  11.  
  12.  
  13.     The following patch fixes two bugs in the bid function of GB5.2.
  14.  
  15.   o Checks to see if the bid and shipping costs exceed your current funds.
  16.     (before it only checked for the bid > money not (bid + ship) > money)
  17.  
  18.   o Checks to see if you have outstanding bids and deducts that amount from
  19.     your available funds.  (to prevent a player from bidding on all the lots
  20.     when he only has the money to pay for some of them)
  21.  
  22.     I highly suggest installing this patch, as the latter of the two bugs 
  23.     can cause core dumps.
  24.  
  25.     Seeker
  26.  
  27.  
  28. -------- build.diffs created on Fri Dec 25 03:16:29 EST 1992 ----------
  29. -------- on machine tensha.dev.rockcoast.com by johnd -----------------
  30.  
  31. *** /work/GB5.2/build.c    Fri Nov 27 14:02:57 1992
  32. --- build.c    Fri Dec 25 03:06:35 1992
  33. ***************
  34. *** 1747,1760 ****
  35.       deductAPs(Playernum, Governor, APcount, snum, 0);
  36.   }
  37.   
  38. ! void bid(int Playernum, int Governor, int APcount)
  39.   {
  40.       racetype *Race;
  41.       planettype *p;
  42.       commodtype *c;
  43.       shiptype *s;
  44.       char commod;
  45.       int i, item, bid0, lot, shipping, ok=0, sh;
  46.       int minbid;
  47.       double dist, rate;
  48.       int snum, pnum;
  49. --- 1747,1763 ----
  50.       deductAPs(Playernum, Governor, APcount, snum, 0);
  51.   }
  52.   
  53. ! void
  54. ! bid(int Playernum, int Governor, int APcount)
  55.   {
  56.       racetype *Race;
  57.       planettype *p;
  58.       commodtype *c;
  59. +     commodtype *cc;
  60.       shiptype *s;
  61.       char commod;
  62.       int i, item, bid0, lot, shipping, ok=0, sh;
  63. +     int u, shipcost = 0, money_owed = 0;
  64.       int minbid;
  65.       double dist, rate;
  66.       int snum, pnum;
  67. ***************
  68. *** 1825,1831 ****
  69.           return;
  70.       }
  71.       getplanet(&p, snum, pnum);
  72. ! 
  73.       if(p->slaved_to && p->slaved_to!=Playernum) {
  74.           sprintf(buf, "This planet is enslaved to player %d.\n", p->slaved_to);
  75.           notify(Playernum, Governor, buf);
  76. --- 1828,1836 ----
  77.           return;
  78.       }
  79.       getplanet(&p, snum, pnum);
  80. !     /*
  81. !      * Cant bid from an enslaved planet 
  82. !      */
  83.       if(p->slaved_to && p->slaved_to!=Playernum) {
  84.           sprintf(buf, "This planet is enslaved to player %d.\n", p->slaved_to);
  85.           notify(Playernum, Governor, buf);
  86. ***************
  87. *** 1832,1859 ****
  88.           free(p);
  89.           return;
  90.       }
  91. ! /* check to see if there is an undamaged gov center or space port here */
  92.       sh = p->ships;
  93. !     while(sh && !ok) {
  94. !     (void)getship(&s, sh);
  95. !     if(s->alive && (s->owner==Playernum) && !s->damage &&
  96. !        Shipdata[s->type][ABIL_PORT]) ok = 1;
  97. !     sh = s->nextship;
  98. !     free(s);
  99.       }
  100.       if(!ok) {
  101. !     notify(Playernum, Governor, "You don't have an undamaged space port or government center here.\n");
  102. !     free(p);
  103. !     return;
  104.       }
  105.   
  106.       lot = atoi(args[1]);
  107.       bid0 = atoi(args[2]);
  108.       if((lot<=0) || lot>Numcommods()) {
  109.           notify(Playernum, Governor, "Illegal lot number.\n");
  110.           free(p);
  111.           return;
  112.       }
  113.       getcommod(&c, lot);
  114.       if(!c->owner) {
  115.           notify(Playernum, Governor, "No such lot for sale.\n");
  116. --- 1837,1875 ----
  117.           free(p);
  118.           return;
  119.       }
  120. !     /*
  121. !      * check to see if there is an undamaged gov center 
  122. !      * or space port here 
  123. !      */
  124.       sh = p->ships;
  125. !     while (sh && !ok) {
  126. !         (void)getship(&s, sh);
  127. !         if (s->alive && (s->owner==Playernum) && !s->damage &&
  128. !            Shipdata[s->type][ABIL_PORT]) 
  129. !                 ok = 1;
  130. !         sh = s->nextship;
  131. !         free(s);
  132.       }
  133. + 
  134.       if(!ok) {
  135. !         notify(Playernum, Governor,
  136. !          "You don't have an undamaged space port or government center here.\n");
  137. !         free(p);
  138. !         return;
  139.       }
  140.   
  141.       lot = atoi(args[1]);
  142.       bid0 = atoi(args[2]);
  143. + 
  144.       if((lot<=0) || lot>Numcommods()) {
  145.           notify(Playernum, Governor, "Illegal lot number.\n");
  146.           free(p);
  147.           return;
  148.       }
  149. + 
  150. +     /* 
  151. +      * What the HELL does this do?! 
  152. +      */
  153.       getcommod(&c, lot);
  154.       if(!c->owner) {
  155.           notify(Playernum, Governor, "No such lot for sale.\n");
  156. ***************
  157. *** 1861,1866 ****
  158. --- 1877,1883 ----
  159.           free(c);
  160.           return;
  161.       }
  162. + /*    
  163.       if (c->owner == Playernum &&
  164.           (c->star_from != Dir[Playernum-1][c->governor].snum ||
  165.            c->planet_from != Dir[Playernum-1][c->governor].pnum)) {
  166. ***************
  167. *** 1869,1876 ****
  168.           free(c);
  169.           return;
  170.       }
  171.       minbid = (int)((double)c->bid*(1.0+UP_BID));
  172. !     if(bid0 < minbid) {
  173.           sprintf(buf, "You have to bid more than %d.\n", minbid);
  174.           notify(Playernum, Governor, buf);
  175.           free(p);
  176. --- 1886,1894 ----
  177.           free(c);
  178.           return;
  179.       }
  180. + */
  181.       minbid = (int)((double)c->bid*(1.0+UP_BID));
  182. !     if(bid0 <= minbid) {
  183.           sprintf(buf, "You have to bid more than %d.\n", minbid);
  184.           notify(Playernum, Governor, buf);
  185.           free(p);
  186. ***************
  187. *** 1877,1882 ****
  188. --- 1895,1904 ----
  189.           free(c);
  190.           return;
  191.       }
  192. + 
  193. +     /* 
  194. +      * Don't allow guests to screw things up 
  195. +      */
  196.       Race = races[Playernum-1];
  197.       if(Race->Guest) {
  198.           notify(Playernum, Governor, "Guest races cannot bid.\n");
  199. ***************
  200. *** 1884,1889 ****
  201. --- 1906,1914 ----
  202.           free(c);
  203.           return;
  204.       }
  205. +     /*
  206. +       * Make sure they have the bucks 
  207. +      */
  208.       if(bid0 > Race->governor[Governor].money) {
  209.           notify(Playernum, Governor, "Sorry, no buying on credit allowed.\n");
  210.           free(p);
  211. ***************
  212. *** 1890,1906 ****
  213.           free(c);
  214.           return;
  215.       }
  216. ! /* notify the previous bidder that he was just outbidded */
  217.       if(c->bidder) {
  218. !         sprintf(buf, "The bid on lot #%d (%d %s) has been upped to %d by %s [%d].\n", lot, c->amount, Commod[c->type], bid0, Race->name, Playernum);
  219.           notify((int)c->bidder, (int)c->bidder_gov, buf);
  220.       }
  221.       c->bid = bid0;
  222.       c->bidder = Playernum;
  223.       c->bidder_gov = Governor;
  224.       c->star_to = snum;
  225.       c->planet_to = pnum;
  226. !     shipping = shipping_cost((int)c->star_to, (int)c->star_from, &dist, (int)c->bid);
  227.   
  228.       sprintf(buf, "There will be an additional %d charged to you for shipping costs.\n", shipping);
  229.       notify(Playernum, Governor, buf);
  230. --- 1915,1973 ----
  231.           free(c);
  232.           return;
  233.       }
  234. !     /* 
  235. !      * Make sure we check to see if we have enough for shipping costs 
  236. !      * as well.    --JPD--
  237. !      */
  238. !     shipcost = shipping_cost((int)c->star_to, (int)c->star_from, 
  239. !                 &dist, (int)c->bid);
  240. ! 
  241. !     if ((bid0 + shipcost) > Race->governor[Governor].money) {
  242. !       sprintf(buf, "The bid (%d) + shipping costs (%d) exceed your money (%d)\n",     
  243. !         bid0, shipcost, Race->governor[Governor].money);
  244. !         notify(Playernum, Governor, buf);
  245. !         free(p);
  246. !         free(c);
  247. !         return;
  248. !     }
  249. ! 
  250. !     /* 
  251. !      * Also make sure they have not bid on other things and trying to use 
  252. !      * money earmarked for other lots --JPD--
  253. !      */
  254. ! 
  255. !     for (u = 1; u <= Numcommods(); u++) {
  256. !         getcommod(&cc, u);
  257. !             if (cc->bidder == Playernum)
  258. !                 money_owed += cc->bid;
  259. !     }
  260. !     free(cc);
  261. !     
  262. !     if (((bid0 + shipcost) + money_owed) > Race->governor[Governor].money) {
  263. !         notify(Playernum, Governor, "All your money is used in bids already!\n");
  264. !         free(p);
  265. !         free(c);
  266. !         return;
  267. !     }
  268. ! 
  269. !     /* 
  270. !      * They passed the credit check, notify the previous bidder 
  271. !      * that he was just outbidded 
  272. !      */
  273.       if(c->bidder) {
  274. !       sprintf(buf, "The bid on lot #%d (%d %s) has been upped to %d by %s [%d].\n", lot, c->amount, Commod[c->type], bid0, Race->name, Playernum);
  275.           notify((int)c->bidder, (int)c->bidder_gov, buf);
  276.       }
  277. + 
  278. +     /* 
  279. +      * Set the commod structs to the new values 
  280. +      */
  281.       c->bid = bid0;
  282.       c->bidder = Playernum;
  283.       c->bidder_gov = Governor;
  284.       c->star_to = snum;
  285.       c->planet_to = pnum;
  286. !     shipping = shipcost;
  287.   
  288.       sprintf(buf, "There will be an additional %d charged to you for shipping costs.\n", shipping);
  289.       notify(Playernum, Governor, buf);
  290.