Date: Thu, 13 Jan 94 09:52:45 CST From: csaba AT vuse DOT vanderbilt DOT edu (Csaba A. Biegl) To: djgpp-bounces AT sun DOT soe DOT clarkson DOT edu Subject: Re: Bug in LIBGRX103 ? Cc: djgpp AT sun DOT soe DOT clarkson DOT edu From djgpp-bounces AT sun DOT soe DOT clarkson DOT edu Wed Jan 12 18:29:19 1994 To: djgpp AT sun DOT soe DOT clarkson DOT edu Subject: Bug in LIBGRX103 ? I found a problem with libgrx.a The function GrFilledPolygon shows a strange behavior. I expected the following program to draw a horizontal line. #include #include void main() { int p[2][2]={{300,200},{310,200}}; GrSetMode(GR_default_graphics); GrFilledPolygon(2,p,GrWhite()); getchar(); GrSetMode(GR_default_text); } The result is something like this: \ \ If i give GrFilledPolygon 3 points (2 identical) it works fine: void main() { int p[3][2]={{300,200},{300,200},{310,200}}; GrSetMode(GR_default_graphics); GrFilledPolygon(3,p,GrWhite()); getchar(); GrSetMode(GR_default_text); } Thanks for the report. The following diff (to ...contrib\libgrx\src\scancnvx.c) fixes the problem: ----------------------------------------------------------- *** scancnvx.old Thu Jan 13 09:09:44 1994 --- scancnvx.c Thu Jan 13 09:08:14 1994 *************** *** 50,56 **** } else { rdx = pt[1][0]; ! rdy = pt[1][0]; MOUSE_BLOCK(CURC,ldx,ldy,rdx,rdy); (*borderproc)(ldx,ldy,rdx,rdy,fillarg); } --- 50,56 ---- } else { rdx = pt[1][0]; ! rdy = pt[1][1]; MOUSE_BLOCK(CURC,ldx,ldy,rdx,rdy); (*borderproc)(ldx,ldy,rdx,rdy,fillarg); } ----------------------------------------------------------- For those who don't want to rebuild libgrx.a the following .ZIP file contains the necessary patches both to the source and to the library itself. Instructions: 1) uudecode the attached .ZIP file. 2) unzip it in the GRX base directory. (For example: \djgpp\contrib\libgrx) If you use pkunzip, don't forget the '-d' switch! 3) CD to the newly created .\TMP sub-directory. 4) Type: DO_PATCH at the DOS prompt. This will overwrite "scancnvx.c" in the ..\SRC sub-directory and replace the "scancnvx.o" module in ..\LIB\LIBGRX.A 5) You may now remove the contents of the TMP sub-directory if you wish. NOTE: the above procedure assumes that the GRX directory tree is the same as in the distribution. If you modified it, change the steps in DO_PATCH.BAT appropriately! ---------------------------------------------------------------------- begin 666 grxpatch.zip M4$L#!!0``````/!*+1P````````````````$````=&UP+U!+`P04````"`#V M2BT"YCG5=M;QI'$/Z,)?^! M?IJD:L3!00Q1FT;843&Q'50,B)5.6_=V9W[P5LJTZ3 M\'*SL\_.///,+'E;*IV>0*D$PU:SV^K^];7:TL_&"*UPM4L%\'O]#(:Q0EROX!.ZT!718J-\7[HPZKKP[GV]7DN=/0;X M8[..9'4C AT JG$B?+CJIRN\R>/%BJ"F?(ET.=413&JR3J64UCS#H AT 7$F*)RPA\ M6B.S"K1M&GKKI0QBB_+:HP14,*]ZD]P)P$;<`#6#((SI0+W5XM@- M$WDQ4(&<5G-,6)REV#,Z`^5YV2K*9!W3F1MB5,1J^63< M259WX3I-P8]"6(B-!)2>5!L"$^E>IN6F.X8;&4 AT 4/O37$U]Y%J2C/!E$E)`? M!G,=@4ECA>$ M@'37;PY&[=:XTQQ`?SSH]X97)(>AY.`2,3PN35J4&>$M0V)O*DE+?F1H>'MZ M$&AYX( M6F&PD;M^Z.^+*H AT AG`,[&4\*,J`M)='T<&6V"AP[IR>_'-Z M4N"(H.2O:BZ]U=T2\C?D;[Z(XD;BX>]VKC_EU][U(Y^TXOH2,<34`\D#I_S: MNV@\\-!COU2!NU^*G;L/Y$YCW_;&PZN'ZT[SIL%L%M2L6`S@(]0<>/,&BD3> MV3W]@XL+)C*HU/@IOU8[6*O=.PX$E4K#0`5P?@%U!SA/S;XUG3G4\_$:@X9= MH,Q`H^C3,NL^M=92JP:A,VL&UZ9PV>FU_BRVQH-61I/YY(XO%(JEM*1.,?%( M*I%`?[>?TB@I\L9 M?^Q(ZJ"8N+[T\:Y^=G;&*VBM AT C^*%6W7`-2C1597PFX#*A6J.A6F`=I>/BX; M'T"RX$4M`/U\D3SG2L-GPGGFB,\YZF`M(LDF";Y(34`?J-]YT3DLO<[F/+]- M9V>?$SX2_`N3?I9*J]/N?^ZTNU>V3MPS]&+GE-K'U;0>CUV+I7R7.\4\WJ.* MOJB2'."P-QA=?;KL?7WR>$HGZ]SGH[1NNIP5*F=2=VC0WXP0&CQ,WZJ>XUC7 MH*(E<&X$XH"?%N30L<83X)5&T/7 AT R6"^LYYX;(9WCNU4HU+>8UO)-+AUKX!! MTRLL.8-]GH`8##Z9?C1VKG0#:5U6-`(%Z193*+LGZW^=JSV?G:',0R=;.HS$ M;->K7)3"!*7X.]^9WY]F$S6;>,@F:]DTE)&[8]6].NH(B$32;>,PFYME,SK=L8H[-HTC^+YLL`D+ZR/>% M$9"^UO24F.9*6>'*.O"6JYBJQ[@2@:_R][T#1U-8T^<:K;D&SS4R..A??3'R MH;]<\"G<&!P4F&/HPM-76%DO-G1.!7,]FV`I";Y3)`QVHSQ MB8SU+Y&T]?*#7B]\S'HA\<1G/?70*R1^^F8H0RWQW"[H/P+F6C AT W8);C%P]Q M(P*39(5+O;.LT(YRV:C.!B"YF:7B%55U,7$44GKM[%]9VZY*V&*I- AT PD&R$9BJ AT W>!Y,&,L$2 M5U=P*-;0+5U6^H-`8&^!Q+N0W&SB=6I9XY/&:'CPP:?&BK'&VJRXRM:@H-(;(UA/',W-UQ$:J3W)EOOG/FG#-GSMQY3G&B<>7$XR.5"'D1 M;ZK2%`F/1]`_[9P+H:LPYESVO!Z^6OB:^GHCO47Y.1?:LYWD>N='1]'_Z$W! M1ZQ?S5SWZ:Y.ZG?3GL/F/25:0XF''J38H[V4*NC[:$`%L*4KI\Q[#N_T1[`H M=L=[[0 AT 8!1V*++S*%W2H%LG"[@K8ZGM:ZF+ZLBVFN(8M M`KEV"0S.XQ0R^)"SAR0R&9O':3Z`"XODU]LIR5&,`FU3+0QNDB?/N4Q.?`J_AE#?V!81'25HC MN:B'XIR%9RT"/9C+:=5U(/1>S3AY\M*:ZRB?3O_`IR2E53<(Z6V8:M6U0G1# M:.9LS>@#M)J/%@\;5G-C!G!U AT DN7I^P8TU-,3XL^=Y\#L!7`U%&>M:JUIT08 M,LCFG4$V[`SRB)`F-9<=Y[[26A&G>T:$"1GE"VIMJE90*9X:X<%PSXC`>6:2 MMEF@&@25_-=>DO^Q%TA_(4Y)P2);YAST!5ZZH?]9[_7LK M]>ZFD1657W`Z1.7-V2/;N M7Q;);/9D8W]&GZ>5/IP&Q\+](I@^)DS7\*(94UDSQ1M%^S5L2=@5JF09S+J9 M?U'Y`["5OUNP\,+F+Y`B>D;5 AT NJDDP95K2LXTX0\ER(UF8J&`XW:@O4+RB\,I:5$B:X936DWK],_ON&3H_YS.J MKL'?!+QOF0GH"W#;M46H!YP7DSV/':9:MQI)09S>V#O;C.UQEN*?,`9K-GS# M?$.1S-I%KCEE)%#$X[V-$X%B8;X5?O^5L\$[W\3C+\-\/WP5*`R]1Z!+T!\0 MZ$,IO2ZELU(ZK)2X28FN2?2>1!]+](E$7TGTK41+2LG;;Y);5TK>JATEKDZB M)R5JD>A%B;HEZI/HLD1C$D4E>E.BMR7ZP%&*ZH;D/G>4HMJ0W"%G"35*-"#1 MI$33$@7 AT B7I0H*97+PZ$BZ_>-D.H7QD-]0Z&!J^,-X5065.+HR)6\79H%_-8 M&>-`Y>\V9YP*?VWKRXWN>+,=0@?M:N7OM?,^.KP]788=HF_;Q73N8L[O8+BC M_E#H>#`T]-HP)*:O"06#P?Y!71+!$#!M(YV0I-:AP2OA\<#0P(2 AT _$/Z:+CU M0CAT64Q!*A(@12T#0T61F))!F_@;4$L#!!0````(`+%++1RAA^#D3````(L` M```0````=&UP+V1O7W!A=&-H+F)A=$O.+ZA4*$Y.S$O.*ZO02U;0TXLI+DKF MY4HL4M"M*`-Q