Opened 9 years ago
Last modified 3 years ago
#3572 accepted defect
Clarifications in the Compiler Flag Documentation
Reported by: | Francesco Casella | Owned by: | Lennart Ochel |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | *unknown* | Version: | |
Keywords: | Cc: | andrea.bartolini@… |
Description
The web page https://openmodelica.org/doc/OpenModelicaUsersGuide/latest/omchelptext.html lists all compiler flags, and is automatically generated from omc help info.
The situation has improved dramatically once #2913 was addressed. However, there are four things that still need to be fixed.
- The documentation gives
-d
and--debug
as debug flags. However, in the page there are still three references to+d
, which are very confusing because it is not clear whether that is a separate flag with different effects or not. Please always use the same notation-d
everywhere. You may also mention in the documentation that the+d
notation is deprecated but can still be found in existing code
- The documentation https://openmodelica.org/doc/OpenModelicaUsersGuide/latest/omchelptext.html#omcflag-debug-section explains how to set up the
-d
flag. However, there are no examples. I would suggest to add one or two, and at least one where multiple flags are set and some are reset, for better clarity.
- The documentation of
--preOptModules
,--initOptModules
and--postOptModules
does not contain any example, so it is not clear how one can actually set the options that are listed there. I would suggest to add a couple of examples from which it is clear what the syntax is.
- The new flags introduced in https://github.com/OpenModelica/OMCompiler/pull/282 are not documented at all, please add the documentation of
--forceRecommendedOrdering
and also add the documentation about+=
and-=
to--preOptModules
, and--postOptModules
. BTW, can this also be used with--initOptModules
, or should a special section dedicated to flags used ininitOptModules
be also introduced in the documentation?
Change History (13)
comment:1 by , 9 years ago
Status: | new → accepted |
---|
follow-up: 3 comment:2 by , 9 years ago
comment:3 by , 9 years ago
Replying to lochel:
Please note that the documentation already contains all the default values. E.g.
--preOptModules
is described here including the default value and all valid options.
This is true, but it is not clear how you apply the string list to the flag - do you need an equality sign? Can you have spaces beween the flag and the string? Has the string to be quoted? A small example would suffice, following the old tradion of man pages.
comment:7 by , 8 years ago
Milestone: | 1.11.0 → 1.12.0 |
---|
Milestone moved to 1.12.0 due to 1.11.0 already being released.
comment:8 by , 7 years ago
Milestone: | 1.12.0 → 1.13.0 |
---|
Milestone moved to 1.13.0 due to 1.12.0 already being released.
comment:10 by , 5 years ago
Milestone: | 1.14.0 → 1.16.0 |
---|
Releasing 1.14.0 which is stable and has many improvements w.r.t. 1.13.2. This issue is rescheduled to 1.16.0
comment:12 by , 4 years ago
Milestone: | 1.17.0 → 1.18.0 |
---|
Retargeted to 1.18.0 because of 1.17.0 timed release.
I will improve the documentation a bit.
Please note that the documentation already contains all the default values. E.g.
--preOptModules
is described here including the default value and all valid options.The flag
--forceRecommendedOrdering
is renamed into--defaultOptModulesOrdering
and briefly described here.Moreover I suggest to remove a bunch of flags, to make the usage of omc more consistent and less complicated (see #3568#comment:9).