I have run the new dm511_bas through my once program, which can identify any possible spelling mistakes. This produced the following output ...
Code: Select all
Candidates for possible spelling mistakes ..
Name(s) occurring once only ..
At line 440 var0448$
At line 2320 row%
At line 2320 col%
At line 4310 var06980
At line 13280 pipe_22
At line 21620 pipe_150
At line 28680 pipe_32000
At line 29250 var0C10
Line 440 seems to be wasting about 9k of memory with an unused string array.
At line 2320, row% and col% are necessary for the return results.
At line 4310, var06980 should be var0698 (from line 4230).
The pipe_22, pipe_150 and pipe_32000 are simply device names.
And at line 29250, var0C10 I guess should really be var0C08 (from the FOR loop at line 29200)
I would also like to point out that at lines 11790, 12050, 12300 and 12580, the variable ad8 has never been assigned a value and so would default to 0, i.e. #0
2390 fra$ = fra$ & procFun25050$((var0658 - 1)) & "_"
or did you really mean to do
2390 fra$ = fra$ & procFun25050$((var0658 - 1),1,LEN(procFun25050$((var0658 - 1)))) & "_"
when the additional parameters will be thrown away.
Lines 16300 and 16340
You've currently got the "NEXT loop" after the "END REPeat loop". Perhaps you should reverse these. But to get it outside of the "END IF"
23680 This line is a bit of a mouthfull. Since there is already an END FOR var03A0 at line 24090, this line could be written more simply as ...
23680 IF (((mark%(var03A0) = 129) OR (mark%(var03A0) = 2)) AND (fil$(var03A0,1) = ">")) THEN var0B28 = var03A0 : ELSE NEXT var03A0 : RETurn
I've never used DiskMate 5, I'm just going on the Basic.