virtlink
6th March 2003 11:53 UTC

About the documentation
Page

(custom function [caption]) | ((license|components|directory|instfiles)
[pre_function] [show_function] [leave_function]) [define_if_last]

I suppose that this means:

(a custom page, etc...) OR (a predefined page, etc...) AND (optional: symbol)

And it could be read as (watch the '(' and ')'s):

((a custom page, etc...) OR (a predefined page, etc...)) AND (optional: symbol)

or:

(a custom page, etc...) OR ((a predefined page, etc...) AND (optional: symbol))

Aren't there some '(' and ')' missing? :)
Same for UninstPage.

By the way, do Page and UninstPage count as an instruction for a relative Goto jump (e.g. Goto +3)?

virtlink
6th March 2003 12:04 UTC

BGGradient

[off]|[topc botc [textcolor|notext]]

It can be read as:
You MAY use 'off', or you MAY use 'topc botc [textcolor|notext]'.
Shouldn't it be:

[off|(topc botc [textcolor|notext])]

You MAY use 'off' or 'topc botc [textcolor|notext]'.

virtlink
6th March 2003 12:09 UTC

CompletedText

[/LANG=lang_id] [Completed text]

Now it says: You MAY use lang_id AND/OR completed_text.
Shouldn't it be:

[[/LANG=lang_id] Completed text]

Since you can't use lang_id without completed_text.

The same for ComponentText. It should be:

[[/LANG=lang_id] text [subtext] [subtext2]]

And DetailsButtonText, DirText, FileErrorText, InstallButtonText, MiscButtonText, SpaceTexts and UninstallButtonText.

virtlink
6th March 2003 12:16 UTC

InstallColors should be:

/windows | (foreground background)

InstProgressFlags should be:

[smooth] [colored]

IfRebootFlag should be:

[jump_if_set [jump_if_not_set]]

IntCmp should be:

val1 val2 jump_if_equal [jump_if_val1_less [jump_if_val1_more]]

Same for IntCmpU.

MessageBox should be:

mb_option_list messagebox_text [return_check jumpto [return_check_2 jumpto_2]]

WriteUninstaller should be:

[Path\]exename.exe

StrCpy should be:

user_var(destination) str [maxlen [start_offset]]

CreateFont should be:

user_var(handle output) face_name [height [weight]] [/ITALIC] [/UNDERLINE] [/STRIKE]

FindWindow has the same kind of error, but I can figure out what it should be.

Joost Verburg
6th March 2003 12:38 UTC

By the way, do Page and UninstPage count as an instruction for a relative Goto jump (e.g. Goto +3)?

These are compiler settings, no run-time instructions.

virtlink
6th March 2003 12:41 UTC

Thanks. Should me mentioned in the docs.

Joost Verburg
6th March 2003 12:47 UTC

Originally posted by virtlink
Thanks. Should me mentioned in the docs.

Maybe it should not be possible to include them in a section or function, just like the other attributes.

virtlink
6th March 2003 12:49 UTC

I agree with that.

The small list of non-runtime commands under the 'Labels' header should also be under the 'Goto' instruction header. Actually, it should be rather there than at 'Labels' since it has nothing to do with labels.

kichik
6th March 2003 16:47 UTC

Page and UninstPage can no longer be used inside sections and functions in latest CVS version. Others will be fixed by flizebogen.

virtlink
6th March 2003 21:11 UTC

KiCHiK's post should be:

Not originally posted by kichik
Page and UninstPage can no longer be used inside sections and functions in latest CVS version. Others will be fixed by flizebogen.

kichik
6th March 2003 21:16 UTC

Happy?

Sunjammer
6th March 2003 23:04 UTC

No, cause virtlinks last post makes no sense to me :P

flizebogen
7th March 2003 11:59 UTC

The changes you requested will be done today night.

virtlink
7th March 2003 12:34 UTC

Sorry Sunjammer. KiCHiK just edited his post. He forgot to type the 'no' word:

Originally posted by kichik
Page and UninstPage can longer be used inside sections and functions in latest CVS version. Others will be fixed by flizebogen

And flizebogen, thanks for applying the requested changes, altough I didn't request them, just suggested and noticed them.