|
Post by mueh on Aug 14, 2023 4:02:43 GMT -5
Hi George ! FM Restore command in Backup folder on Backup File fails with msg "File could not be found??" since V3.23169 . No Problem in V3.23088 and previous versions . Wish you a sunny weather . All 4 cmd's RESTORE RS RESTORET RST in upper and lower case get now msg in 23169 . in 23088 only RESTORE RESTORET fail with msg . Maybe Following code change in 23169 fails to set up lclMode .
METHOD FMLCmdRestore(i AS LONG)
'--------------------------------------------------------------------------------------------------+
'- FM Restore a Backup copy |
'--------------------------------------------------------------------------------------------------+
LOCAL lclFileName, lclMode, ErrMsg AS STRING, j AS LONG
MEntry
IF gFMD(i).CmdRoot = "RS" THEN ' Choose existing or new mode
lclMode = "E" ' Existing
ELSEIF gFMD(i).CmdRoot = "RST" THEN '
lclMode = "T" ' With Timestamp
END IF '
gFMD(i).Cmd = " " ' Clear command char
|
|
|
Post by Robert on Aug 14, 2023 7:37:27 GMT -5
Much, do you notice that lclMode is defined as LONG but string constants are being assigned to it?
Maybe that is the problem?
R
|
|
|
Post by George on Aug 16, 2023 14:22:32 GMT -5
Robert: Before I look at this, note that lclMode is, in fact, defined as STRING.
And now I get to go and re-learn what Backup and Restore really do. Old age sure affects the memory.
George
|
|
|
Post by Robert on Aug 16, 2023 14:24:13 GMT -5
My bad, I have always hated how PB lets you omit types like this, it's just so unreadable (in my eyes).
Sorry, Robert doesn't know what he's talking about.
R
|
|
|
Post by George on Aug 16, 2023 14:37:13 GMT -5
Robert: It may be permitted, but I (and I imagine 99% of programmers) always run with #DIM ALL - meaning EVERYTHING must be defined.
Yes you can use the old BASIC DEFINT type specifiers, but I seriously doubt anyone uses that style anymore.
What you missed was that a PB definition can describe multiple types with one statement.
e.g.
LOCAL A, B, C AS LONG, S, T, U AS STRING, X, Y, Z AS DWORD
George
|
|
|
Post by George on Aug 17, 2023 9:36:27 GMT -5
MUEH: OK, found it. Last release revised all the handling of FM Line commands to clean up block mode, repeats etc. RS and RST were mucked up (and some others functions as well such as issuing FM Line commands from a macro).
George
|
|
|
Post by mueh on Aug 22, 2023 0:50:40 GMT -5
Hi George ! Did RS cmd in 23232 and it overwrites now existing file without any warning . ************Forget this update .Warning is issued correctly . Maybe i deleted file before RS on first test . Thanks
|
|