mirror of
https://git.lyx.org/repos/lyx.git
synced 2024-11-13 22:49:20 +00:00
7867 lines
142 KiB
Plaintext
7867 lines
142 KiB
Plaintext
#LyX 2.4 created this file. For more info see https://www.lyx.org/
|
||
\lyxformat 614
|
||
\begin_document
|
||
\begin_header
|
||
\save_transient_properties true
|
||
\origin /systemlyxdir/doc/
|
||
\textclass scrbook
|
||
\options BCOR8mm,captions=tableheading
|
||
\use_default_options false
|
||
\begin_modules
|
||
logicalmkup
|
||
\end_modules
|
||
\maintain_unincluded_children no
|
||
\language english
|
||
\language_package default
|
||
\inputencoding utf8
|
||
\fontencoding auto
|
||
\font_roman "lmodern" "default"
|
||
\font_sans "lmss" "default"
|
||
\font_typewriter "lmtt" "default"
|
||
\font_math "auto" "auto"
|
||
\font_default_family default
|
||
\use_non_tex_fonts false
|
||
\font_sc false
|
||
\font_roman_osf false
|
||
\font_sans_osf false
|
||
\font_typewriter_osf false
|
||
\font_sf_scale 100 100
|
||
\font_tt_scale 100 100
|
||
\use_microtype false
|
||
\use_dash_ligatures true
|
||
\graphics default
|
||
\default_output_format pdf2
|
||
\output_sync 1
|
||
\bibtex_command default
|
||
\index_command default
|
||
\float_placement class
|
||
\float_alignment class
|
||
\paperfontsize 12
|
||
\spacing single
|
||
\use_hyperref true
|
||
\pdf_title "LyX's Development manual"
|
||
\pdf_author "LyX Team"
|
||
\pdf_subject "LyX's development documentation"
|
||
\pdf_keywords "LyX, Documentation, Development"
|
||
\pdf_bookmarks true
|
||
\pdf_bookmarksnumbered true
|
||
\pdf_bookmarksopen true
|
||
\pdf_bookmarksopenlevel 1
|
||
\pdf_breaklinks false
|
||
\pdf_pdfborder false
|
||
\pdf_colorlinks true
|
||
\pdf_backref false
|
||
\pdf_pdfusetitle false
|
||
\pdf_quoted_options "linkcolor=black, citecolor=black, urlcolor=blue, filecolor=blue, pdfpagelayout=OneColumn, pdfnewwindow=true, pdfstartview=XYZ, plainpages=false"
|
||
\papersize a4
|
||
\use_geometry false
|
||
\use_package amsmath 1
|
||
\use_package amssymb 1
|
||
\use_package cancel 1
|
||
\use_package esint 1
|
||
\use_package mathdots 1
|
||
\use_package mathtools 1
|
||
\use_package mhchem 1
|
||
\use_package stackrel 1
|
||
\use_package stmaryrd 1
|
||
\use_package undertilde 1
|
||
\cite_engine basic
|
||
\cite_engine_type default
|
||
\biblio_style plain
|
||
\use_bibtopic false
|
||
\use_indices false
|
||
\paperorientation portrait
|
||
\suppress_date false
|
||
\justification true
|
||
\use_refstyle 0
|
||
\use_minted 0
|
||
\use_lineno 0
|
||
\notefontcolor #0000ff
|
||
\index Index
|
||
\shortcut idx
|
||
\color #008000
|
||
\end_index
|
||
\secnumdepth 4
|
||
\tocdepth 2
|
||
\paragraph_separation indent
|
||
\paragraph_indentation default
|
||
\is_math_indent 0
|
||
\math_numbering_side default
|
||
\quotes_style english
|
||
\dynamic_quotes 0
|
||
\papercolumns 1
|
||
\papersides 2
|
||
\paperpagestyle headings
|
||
\tablestyle default
|
||
\tracking_changes false
|
||
\output_changes false
|
||
\change_bars false
|
||
\postpone_fragile_content false
|
||
\html_math_output 0
|
||
\html_css_as_file 0
|
||
\html_be_strict true
|
||
\docbook_table_output 0
|
||
\docbook_mathml_prefix 1
|
||
\end_header
|
||
|
||
\begin_body
|
||
|
||
\begin_layout Title
|
||
Developing \SpecialChar LyX
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Subtitle
|
||
Version 2.4.x
|
||
\end_layout
|
||
|
||
\begin_layout Author
|
||
by the \SpecialChar LyX
|
||
Team
|
||
\begin_inset Foot
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
If you have comments on or error corrections to this documentation,
|
||
please send them to the \SpecialChar LyX
|
||
Documentation mailing list:
|
||
|
||
\begin_inset CommandInset href
|
||
LatexCommand href
|
||
target "lyx-docs@lists.lyx.org"
|
||
type "mailto:"
|
||
literal "false"
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset CommandInset toc
|
||
LatexCommand tableofcontents
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Chapter
|
||
Introduction
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
This manual documents some aspects of \SpecialChar LyX
|
||
development.
|
||
It is currently rather incomplete,
|
||
but will hopefully be extended in the future.
|
||
Meanwhile,
|
||
additional information can be found in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subfolder of the \SpecialChar LyX
|
||
source code distribution.
|
||
This document is not translated,
|
||
since the development language of \SpecialChar LyX
|
||
is English.
|
||
If you just want to use \SpecialChar LyX
|
||
,
|
||
then you don't need to read this manual.
|
||
However,
|
||
if you want to learn more about how \SpecialChar LyX
|
||
is developed,
|
||
or even want to participate in \SpecialChar LyX
|
||
development,
|
||
you may find some interesting information here.
|
||
\end_layout
|
||
|
||
\begin_layout Chapter
|
||
File formats
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\SpecialChar LyX
|
||
uses several custom file formats for configuration files and documents.
|
||
This chapter contains some background concerning these file formats.
|
||
Several file formats are also described in detail in the regular user documentation.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
When is an update of the .lyx file format number needed?
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "sec:When-is-an"
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
When you are working on a new feature you may ask yourself whether it needs an update of the .lyx file format number.
|
||
Whether an update is needed or not is not always obvious.
|
||
Rule of thumb:
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Quote
|
||
Whenever there is the danger that a previous version of LyX cannot open a file using the new feature,
|
||
a file format update is needed.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The file format change allows lyx2lyx rules to implement backwards compatibility.
|
||
Below you can find a list of reasons for file format updates with explanations:
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
New
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
document
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
setting Whenever you introduce a new setting that is stored in the document header,
|
||
a file format update is needed.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
Removed
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
document
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
setting If a certain setting becomes obsolete and gets removed,
|
||
a file format update is needed.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
New
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
valid
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
value
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
for
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
an
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
existing
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
setting,
|
||
e.
|
||
\begin_inset space \thinspace{}
|
||
\end_inset
|
||
|
||
g.
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Description
|
||
\paragraph_spacing single
|
||
Automatically
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
loaded
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
math
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
package The reason for this is that there is no true ERT inset for math formulas:
|
||
Each command is parsed,
|
||
and if a user happens to define a local command with the same name as a command that triggers an automatic load of a package,
|
||
they need to be able to switch off the automatic loading of that package.
|
||
This switch is stored by the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
use_package
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
header setting.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
New
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
language that is stored in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\backslash
|
||
language
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
|
||
\begin_inset Note Note
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
This requirement is under discussion.
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Description
|
||
New
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
inset Of course a new inset requires a file format update.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
New
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
style If a new style or inset layout is added to any layout file or module shipped with \SpecialChar LyX
|
||
,
|
||
then a new file format is needed in the master (development) branch.
|
||
It is possible to backport new styles to the stable version without a file format change.
|
||
See
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "subsec:Backporting-new-styles"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
for more information.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
Removed
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
style If a style or inset layout is removed in any layout file or module shipped with \SpecialChar LyX
|
||
,
|
||
a new file format is required.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
However,
|
||
|
||
\series bold
|
||
new
|
||
\series default
|
||
layouts and modules do
|
||
\series bold
|
||
not
|
||
\series default
|
||
require a file format update (changed 03/16,
|
||
see
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "subsec:New-layouts"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
).
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
If you are still unsure,
|
||
please ask on the development list.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "subsec:update_lyx_files"
|
||
|
||
\end_inset
|
||
|
||
How to update the file format number of .lyx files
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Once you come to the conclusion that a file format update is needed,
|
||
you should use the following procedure to perform the update:
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Implement and test the new feature,
|
||
including the reading and writing of .lyx files.
|
||
Note that any file produced at this stage does not use a valid format,
|
||
so do not use this version of \SpecialChar LyX
|
||
for working on any important documents.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "enu:Describe_format"
|
||
|
||
\end_inset
|
||
|
||
Describe the new format in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/FORMAT
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Update the \SpecialChar LyX
|
||
file format number in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/version.h
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "enu:Add-an-entry"
|
||
|
||
\end_inset
|
||
|
||
Add an entry to both format lists (for conversion and reversion) in
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lib/lyx2lyx/lyx_2_4.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
Add a conversion routine if needed (e.
|
||
\begin_inset space \thinspace{}
|
||
\end_inset
|
||
|
||
g.,
|
||
a new header setting always needs a conversion that adds the new setting,
|
||
but a new document language does not need one).
|
||
Add a reversion routine if needed.
|
||
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
While the conversion routine is required to produce a document that is equivalent to the old version,
|
||
the requirements of the reversion are not that strict.
|
||
If possible,
|
||
try to produce a proper reversion,
|
||
using ERT if needed,
|
||
but for some features this might be too complicated.
|
||
In this case,
|
||
the minimum requirement of the reversion routine is that it produces a valid document which can be read by an older \SpecialChar LyX
|
||
.
|
||
If absolutely needed,
|
||
even data loss is allowed for the reversion.
|
||
(In that case,
|
||
you might want to add a LyX comment that indicates what you have had to do,
|
||
so the user is at least warned).
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Since tex2lyx has several implicit file format dependencies caused by sharing code with \SpecialChar LyX
|
||
,
|
||
updating the file format of .lyx files produced by tex2lyx at the same time as updating the main .lyx file format is strongly recommended.
|
||
Therefore,
|
||
a compiler warning will be issued if the \SpecialChar LyX
|
||
and tex2lyx .lyx file format numbers differ.
|
||
In many cases the tex2lyx update requires only the first and last item of the list below:
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Enumerate
|
||
Update the tex2lyx file format number in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/version.h
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
If the lyx2lyx conversion from the old to the new format is empty,
|
||
or if tex2lyx does not yet output the changed feature,
|
||
you do not need any further tex2lyx changes.
|
||
Otherwise,
|
||
search for the changed feature in tex2lyx,
|
||
and adjust the output according to the lyx2lyx changes.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Update the tex2lyx test references as described in
|
||
\begin_inset CommandInset ref
|
||
LatexCommand formatted
|
||
reference "sec:Updating-test-references"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Enumerate
|
||
If you did not implement full tex2lyx support for the new feature,
|
||
add a line to
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/TODO.txt
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
describing the missing bits.
|
||
Note that it is perfectly fine if you do not add full tex2lyx support for a new feature:
|
||
The updating recommendation above is only issued for the syntax of the produced .lyx file.
|
||
It is no problem if some features supported by \SpecialChar LyX
|
||
are still output as ERT by tex2lyx.
|
||
The problems in the past that resulted in the update recommendation were related to mixed version syntax,
|
||
not ERT.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
It would be nice if you could create a .lyx test file which contains instances of all changed or added features.
|
||
This could then be used to test lyx2lyx and tex2lyx.
|
||
Test samples are collected under the corresponding subdirectories of
|
||
\family typewriter
|
||
/autotests
|
||
\family default
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "enu:updatefiles"
|
||
|
||
\end_inset
|
||
|
||
Test your lyx2lyx code by updating LyX's .lyx documentation files to the new format.
|
||
The developer who makes the change knows best what changes to expect when inspecting the resulting diff.
|
||
Because of this,
|
||
you might be able to catch a bug in the lyx2lyx code that updates the format just by taking a quick scan through the large diff that is the result.
|
||
\begin_inset Note Note
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
Another advantage is that if later we suspect a bug in lyx2lyx we can easily see which layout update made an unexpected change by looking at the git log of a .lyx file that suffers the problem.
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
To do this,
|
||
first make sure that there are no changes to the git repository that you will not want to commit (this is needed because it will be convenient to commit with the command
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
git commit -a
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
).
|
||
Then run the following command in the root folder of the source:
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
python development/tools/updatedocs.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
Look at the resulting changes using the command
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
git diff
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
If anything looks surprising,
|
||
please investigate.
|
||
Keep in mind that the case of
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
LFUNs.lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
is special,
|
||
because it is first generated with
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
gen_lfuns.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
before being converted to the latest format.
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
|
||
\begin_inset Note Greyedout
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\series bold
|
||
Note:
|
||
|
||
\series default
|
||
Only commit file format changes in the doc files if these files are using the new feature of the new file format.
|
||
The reason is rule
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "enu:The-fileformat-of"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
of the documentation policies described in sec.
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "sec:Documentation-policies"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Finally,
|
||
commit using
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
git commit -a
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Updating the file format number of layout files
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The procedure for updating the layout files is similar to that in step
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "enu:updatefiles"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
in section
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "subsec:update_lyx_files"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
One need only run
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
python development/tools/updatelayouts.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
instead of
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
updatedocs.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Note that we do not automatically update any local layout used in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
.lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
files shipped with \SpecialChar LyX
|
||
because users would then not be able to export to older formats.
|
||
For example,
|
||
if a 2.2.0 user exported a template to 2.1.x format and tried to open the file in \SpecialChar LyX
|
||
2.1.x,
|
||
there would be an error because the file would contain a local layout whose format is too new.
|
||
The root reason for this is that we do not support converting layouts to older layout formats,
|
||
as we do for the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
.lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
file format.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Backporting new styles to the stable version
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "subsec:Backporting-new-styles"
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Starting with the stable \SpecialChar LyX
|
||
2.1 branch,
|
||
there is a mechanism in place to backport new styles to the stable version without the need to update the file format.
|
||
The basic idea is that the new style definition is automatically copied to the document preamble so that it can even be used by older minor versions that did not yet include the style.
|
||
To backport a new style to the stable version,
|
||
the following steps are needed:
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Add the line
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ForceLocal -1
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
to the style definition in the development version.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Copy the style definition to the stable version,
|
||
but use
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ForceLocal 1
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
instead.
|
||
If needed adjust the format to the one used by the stable version (see the customization manual for details of the layout file format).
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
For each update of the style in a later stable version,
|
||
increase the argument of
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ForceLocal
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
by one.
|
||
(In the stable version,
|
||
the development version should not be touched.)
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
For details about the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ForceLocal
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
flag see the customization manual.
|
||
No
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lyx2lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
support is needed for backported styles:
|
||
Since the style of the development version has an infinite version number,
|
||
it will always be used.
|
||
Furthermore,
|
||
since its version number is less than one,
|
||
the style will not be written anymore to the document header for files saved by the new version.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Updating the file format number of bind/ui files
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
A change to the functionality of existing LFUNs can require a conversion of
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
.bind
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
and
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
.ui
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
files,
|
||
and therefore an increment of the LFUN format,
|
||
as well as a conversion of Info insets in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
.lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
files for manuals.
|
||
The latter cannot be done automatically and also requires an update of the \SpecialChar LyX
|
||
file format.
|
||
(Think e.g.
|
||
\begin_inset space \space{}
|
||
\end_inset
|
||
|
||
of someone who might have made a set of \SpecialChar LyX
|
||
teaching manuals for use in their own group.)
|
||
\begin_inset Foot
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
\begin_inset Flex URL
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
https://www.lyx.org/trac/ticket/9794
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
To update the LFUN format:
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Increment the LFUN file format number in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/LyXAction.h
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Implement the LFUN conversion in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lib/scripts/prefs2prefs_lfuns.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
See step
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "enu:updatefiles"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
in section
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "subsec:update_lyx_files"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
but instead of the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
updatedocs.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
command,
|
||
use this command:
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
bash development/tools/updatelfuns.sh
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
|
||
\begin_inset Note Note
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
This file should really be converted to python.
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Update Info insets in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
.lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
files.
|
||
To do so,
|
||
increment the \SpecialChar LyX
|
||
format and proceed as in
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "subsec:update_lyx_files"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
,
|
||
steps
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "enu:Describe_format"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
–
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "enu:updatefiles"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
In the lyx2lyx implementation (step
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "enu:Add-an-entry"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
),
|
||
implement a conversion similar to the one in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
prefs2prefs_lfuns.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
above,
|
||
as well as a corresponding reversion;
|
||
for this one can use
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
convert_info_insets
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
from
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lib/lyx2lyx/lyx2lyx_tools.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Chapter
|
||
New layouts and modules
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "subsec:New-layouts"
|
||
|
||
\end_inset
|
||
|
||
New layouts
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Adding a new layout file to the \SpecialChar LyX
|
||
library makes it an
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
officially supported
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
layout.
|
||
You should therefore think carefully about whether you really want to do this and discuss it on lyx-devel,
|
||
since you will need to be prepared to update and fix the layout if necessary.
|
||
If the layout is experimental or for a rarely used document class,
|
||
then it may be better to add it to the relevant portion of the \SpecialChar LyX
|
||
wiki,
|
||
as a user contribution.
|
||
See
|
||
\begin_inset CommandInset href
|
||
LatexCommand href
|
||
target "https://wiki.lyx.org/Layouts/Layouts"
|
||
literal "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
In older versions of this document,
|
||
it was stated that new layout files require a file format change.
|
||
After some discussion,
|
||
it was decided that this is not needed.
|
||
\begin_inset Foot
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
See
|
||
\begin_inset CommandInset href
|
||
LatexCommand href
|
||
name "the thread “Proposal for a guide on updating layouts”"
|
||
target "http://permalink.gmane.org/gmane.editors.lyx.devel/161202"
|
||
literal "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Quote
|
||
For reference,
|
||
here are the arguments on each side
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Description
|
||
Pro
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
New layout files are a file format change
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
All documents produced by 2.2.
|
||
\begin_inset Formula $x$
|
||
\end_inset
|
||
|
||
can always be edited and exported even if
|
||
\begin_inset Formula $x$
|
||
\end_inset
|
||
|
||
is different.
|
||
This is important for people using different machines,
|
||
or exchanging work with colleagues.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
Con
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
New layout files are not a file format change
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
No new LaTeX classes can be supported in a stable version,
|
||
and stable versions have a typical lifetime of 2–3 years.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
We have the same situation already with custom layout files:
|
||
If a document using a custom layout file is moved between machines or people,
|
||
then the layout file needs to be exchanged as well.
|
||
If that is not done,
|
||
then we have a fallback implemented so that such documents can still be edited,
|
||
but not exported,
|
||
and the user gets a warning.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
The lyx2lyx script cannot do anything useful in such a case.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Standard
|
||
If you have decided that you are going to add a new layout file to \SpecialChar LyX
|
||
itself,
|
||
then,
|
||
you should do the following:
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Put your new layout file in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lib/layouts/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
and add it to Git (
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
git add lib/layouts/newlayout.layout
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
) so that it will be committed.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Add an entry in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lib/Makefile.am
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
,
|
||
so that the new layout actually gets installed.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Add an entry in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lib/doc/LaTeXConfig.lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
containing in particular a line like
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Quote
|
||
Found:
|
||
[InsetInfo]
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
where [InsetInfo] is obtained by entering in the minibuffer (Alt+X)
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
info-insert textclass <name>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
This inset will automatically display a boxed
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
yes
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
or
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
no
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
depending on the availability of the package.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Enumerate
|
||
A template or example is strongly encouraged (but not necessarily required).
|
||
It is also possible to provide both.
|
||
Add them to
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lib/templates/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
or
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lib/examples/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
,
|
||
respectively.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Reconfigure \SpecialChar LyX
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Ensure the autotests for the new layout pass (see
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "par:when-to-run-an-export-test"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
).
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
New modules
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Adding a new module is very similar to adding a new layout.
|
||
Therefore,
|
||
the previous section applies to new modules as well,
|
||
with two exceptions:
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
You only need to add an entry to
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lib/doc/LaTeXConfig.lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
if the module requires a LaTeX package.
|
||
In that case,
|
||
the command for entering the InsetInfo is:
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
info-insert package <name>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Modules do not need a template,
|
||
only an example,
|
||
which is strongly encouraged but not necessarily required.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Layouts for document classes with incompatible versions
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset Note Greyedout
|
||
status open
|
||
|
||
\begin_layout Description
|
||
Note:
|
||
This section is currently only a proposal under discussion.
|
||
Please correct/amend as suited.
|
||
Remove this note once a consensus is found.
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
See the thread
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
Proposal for a guide on updating layouts
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
for details and background
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
http://permalink.gmane.org/gmane.editors.lyx.devel/161126
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Every now and then,
|
||
there are changes to LaTeX document classes that break backwards compatibility.
|
||
\begin_inset Foot
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
Uwe has suggested we implement automatic detection of changes in class files.
|
||
This could be done by running a script every month that checks if a document class was changed at CTAN and at the homepages of the scientific journals.
|
||
If it reports a change,
|
||
we can check if our template and layout file are still usable with the changed document class.
|
||
(This is different from the autotests insofar,
|
||
as this would also catch changes that do not result in compilation errors.)
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
Reasons can be a new name for the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
*.cls
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
file,
|
||
removed \SpecialChar LaTeX
|
||
commands,
|
||
or both.
|
||
How should this best be handled in \SpecialChar LyX
|
||
?
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The idea is to support the new version with a new \SpecialChar LyX
|
||
layout so that:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Existing documents can still be opened in \SpecialChar LyX
|
||
and will continue to work on systems where the old version is still installed.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
With differently named
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
*.cls
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
files,
|
||
\SpecialChar LyX
|
||
can check for the availability of the particular version and reflect this in the GUI.
|
||
Different document class versions with the same file name are currently (2.2.x) not detected by the configuration script.
|
||
This is planned for 2.3.
|
||
\begin_inset Foot
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg192467.html
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
However,
|
||
what we really need is version detection for the configuration,
|
||
so that the user can be warned if the required class file has the wrong version.
|
||
(If the class file keeps the name over the version change,
|
||
only one of the two layout files generates compilable documents.)
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
This point was also made here:
|
||
http://permalink.gmane.org/gmane.editors.lyx.devel/143798
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
The new layout can be added both to the master and the stable branches,
|
||
in accord with the policy discussed in
|
||
\begin_inset CommandInset ref
|
||
LatexCommand formatted
|
||
reference "subsec:New-layouts"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
No lyx2lyx conversion is then required when a new major version is released.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The user can move an existing document to the new version simply by selecting a new document class.
|
||
This step is well supported by \SpecialChar LyX
|
||
,
|
||
with established methods for handling unsupported styles and other changes.
|
||
This way,
|
||
no lyx2lyx code is required.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The steps to support a new version of an existing document class are thus:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Create a new layout file including the upstream version in the name (avoid special characters like spaces and dots),
|
||
e.g.
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
acmsiggraph-v0-92.layout
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Include the name of the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
*.cls
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
file as an optional argument in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\backslash
|
||
DeclareLaTeXClass
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
line and include the version number in the GUI name:
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\backslash
|
||
DeclareLaTeXClass[acmsiggraph]{ACM SIGGGRAPH (v.
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
0.92)}
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Update the GUI name in the old layout file (whose name should not be changed),
|
||
e.g.:
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\backslash
|
||
DeclareLaTeXClass{ACM SIGGRAPH (<= v.
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
0.91,
|
||
obsolete)}
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
To avoid duplicate definitions,
|
||
the new layout can
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
Input
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
the old layout file and add\SpecialChar breakableslash
|
||
remove\SpecialChar breakableslash
|
||
obsolete\SpecialChar breakableslash
|
||
modify settings and styles (similar to the inclusion of
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
*.inc
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
files).
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
It may be tempting to let the new layout be the
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
master version
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
and have the old layout import it.
|
||
However,
|
||
this should not be done because any changes to the new layout would need undo steps in the importing old layout.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
If the new LaTeX document class obsoletes the old one,
|
||
update the example and template files to use the new layout.
|
||
Add a note about the changes (preferably with a pointer to the documentation of the changes).
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
This way,
|
||
new documents based on the template or example will use the up-to-date document class version.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Standard
|
||
\begin_inset Newpage newpage
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Chapter
|
||
Tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Automated tests are an important tool to detect bugs and regressions in software development.
|
||
Some projects like gcc even require each bug fix to be accompanied by a test case for the automatic test suite,
|
||
that would detect this bug.
|
||
Testing interactive features automatically is of course very hard,
|
||
but core functionality like document import and export can be tested quite easily,
|
||
and some tests of this kind exist.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
unit tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
There are attempts to set up a suite of unit tests for LyX.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
TODO:
|
||
describe what is done and what is still to do.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
tex2lyx tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The tex2lyx tests are located in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/test
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subfolder of the \SpecialChar LyX
|
||
source code distribution.
|
||
The actual testing is performed by the simple python script
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/test/runtests.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
Each test consists of two files:
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
<test name>.tex
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
contains the \SpecialChar LaTeX
|
||
code that should be tested.
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
<test name>.lyx.lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
contains the expected output of tex2lyx.
|
||
When a test is run,
|
||
the actual produced output is compared with the stored reference output.
|
||
The test passes if both are identical.
|
||
The test machinery is also able to generate a file
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
<test name>.lyx.tex
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
by exporting the produced .lyx file with \SpecialChar LyX
|
||
again.
|
||
This may be useful for roundtrip comparisons.
|
||
\end_layout
|
||
|
||
\begin_layout Subsection
|
||
Running the tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The tex2lyx tests can be run in several ways.
|
||
When in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subfolder of the build directory,
|
||
the commands
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
(cmake,
|
||
all platforms),
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
make test
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
(cmake,
|
||
when using a make based build system and not MSVC) or
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
make alltests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
(autotools) will run the tex2lyx tests.
|
||
Alternatively,
|
||
in the root of the build directory,
|
||
the command
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -R tex2lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
runs all tests whose names match the regex
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
tex2lyx
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
.
|
||
Another way to run the tex2lyx tests in the root build directory is to instead use the command
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -L '(cmplyx|roundtrip)'
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
,
|
||
which runs all tests categorized with the label
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
roundtrip
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
or
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
cmplyx
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
.
|
||
If a test fails,
|
||
the differences between the expected and actual results are output in unified diff format.
|
||
\end_layout
|
||
|
||
\begin_layout Subsection
|
||
Updating test references
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "sec:Updating-test-references"
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
In some cases a changed tex2lyx output is not a test failure,
|
||
but wanted,
|
||
e.
|
||
\begin_inset space \thinspace{}
|
||
\end_inset
|
||
|
||
g.
|
||
\begin_inset space \space{}
|
||
\end_inset
|
||
|
||
if a tex2lyx bug was fixed,
|
||
or a new feature was added.
|
||
In these cases the stored references need to be updated.
|
||
To do so if using autotools,
|
||
call
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
make updatetests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subdirectory of the build directory.
|
||
If instead using CMake,
|
||
call
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
make updatetex2lyxtests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
in the build directory or in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/test
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subdirectory of the build directory.
|
||
\begin_inset Foot
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
Note that this is a case where a make target in the build directory can affect the source directory,
|
||
which might not be advisable.
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
On Windows do the following:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Assure that the path to the python.exe is in your system PATH variable.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Double-click on the file
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
updatetex2lyxtests.vcxproj
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
in the build directory or in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/test
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subdirectory of your build directory.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
In the appearing MSVC program assure that you build the
|
||
\emph on
|
||
Release
|
||
\emph default
|
||
version,
|
||
then right-click on the project
|
||
\family sans
|
||
updatetex2lyxtests
|
||
\family default
|
||
in the project explorer and choose then
|
||
\family sans
|
||
Project
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
Only\SpecialChar menuseparator
|
||
Rebuild
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
only
|
||
\family default
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
For convenience,
|
||
these commands also produce re-exported roundtrip .lyx.tex files.
|
||
Please examine the changed output carefully before committing the changed files to the repository:
|
||
Since the test machinery does not do a roundtrip test .tex
|
||
\begin_inset Formula $\Rightarrow$
|
||
\end_inset
|
||
|
||
.lyx
|
||
\begin_inset Formula $\Rightarrow$
|
||
\end_inset
|
||
|
||
.tex,
|
||
and does not compare the produced dvi or pdf output,
|
||
it assumes that the stored .lyx reference produces correct output if processed by \SpecialChar LyX
|
||
.
|
||
There is only one chance to detect wrong output:
|
||
before committing a new reference.
|
||
Once it is committed,
|
||
it is quite difficult to verify whether it is correct.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Please
|
||
\emph on
|
||
do not
|
||
\emph default
|
||
update the test references by opening them with \SpecialChar LyX
|
||
or directly running lyx2lyx on them.
|
||
This would not work,
|
||
since lyx2lyx and \SpecialChar LyX
|
||
produce slightly different files regarding insignificant whitespace and line breaks.
|
||
\end_layout
|
||
|
||
\begin_layout Subsection
|
||
Adding a new test
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
In many cases tests for new features may be added to one of the existing test files,
|
||
but sometimes this is not possible or not wanted.
|
||
Then a new test file needs to be added:
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Create the new file
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/test/<test name>.tex
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
and run tex2lyx in roundtrip mode to produce the file
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/test/<test name>.lyx.lyx
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
This file will be the new reference.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Once you confirmed that the tex2lyx output is correct,
|
||
add the new files to the corresponding lists in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/test/runtests.py
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
,
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/Makefile.am
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
and
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/tex2lyx/test/CMakeLists.txt
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Commit the changes to the repository,
|
||
or send a patch to the development list and ask for committing if you do not have commit rights.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
ctest automatic tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Some tests are located in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/autotests/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subfolder of the \SpecialChar LyX
|
||
source code distribution.
|
||
|
||
\begin_inset Foot
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
The README document in this folder only describes the
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
keytests
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
subset of autotests!
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
These tests can be run by the commands
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
in the
|
||
\emph on
|
||
build directory
|
||
\emph default
|
||
(all platforms) or (when using a make based build system and not MSVC)
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
make test
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
autotests/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subfolder of the
|
||
\emph on
|
||
build directory
|
||
\emph default
|
||
.
|
||
The test logs are written to the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
Testing/Temporary/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subfolder of the
|
||
\emph on
|
||
|
||
\emph default
|
||
build directory.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Subsection
|
||
Export tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The export tests are integration tests.
|
||
They take longer to run and are more likely to break than the tex2lyx tests.
|
||
Nevertheless,
|
||
they have caught many regressions and without a better alternative it is important to keep them up-to-date and understand how they work.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The export tests
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
reuse
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
documentation,
|
||
template,
|
||
and example documents.
|
||
In addition,
|
||
there are a number of dedicated sample documents in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
autotests/export/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subfolder of the \SpecialChar LyX
|
||
source code distribution.
|
||
All samples are (after copying and eventual processing by scripts) exported to various output formats via the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
—
|
||
export-to
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
command line option.
|
||
The tests checks for errors reported by LyX.
|
||
(However,
|
||
error-free export is no guarantee for an error-free output document.)
|
||
\end_layout
|
||
|
||
\begin_layout Subsubsection
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "par:when-to-run-an-export-test"
|
||
|
||
\end_inset
|
||
|
||
Expectations of LyX developers
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Because the export tests are integration tests and take a long time to run,
|
||
LyX developers are rarely expected to run all of the tests.
|
||
Here are some good practices to follow by developers:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
When making a non-trivial change to a .layout file,
|
||
run the export and layout tests corresponding with that .layout file.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
When making non-trivial changes to a .lyx file,
|
||
run the export tests corresponding to that .lyx file.
|
||
|
||
\begin_inset Foot
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
This rule is due to revision.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
There is an objection from the documentation maintainer that working on the documentation must not be complicated by having to consider non-standard exports.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
successful compiling/testing an edited documentation file with pdflatex suffices to ensure it can be commited,
|
||
not tests with other exports are required.
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
If sudden failures with other exports due to “half-tested” documentation updates are a problem for the test maintainer,
|
||
the test suite should use copies that are
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
copied to a cache dir (autotests/samples/doc/,
|
||
say) but not changed,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
updated regularely (but on a time chosen by the test suite maintainer) from the originals in lib/doc/
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
This way,
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
no test will fail due to ongoing work on documentation,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
the documentation is still tested in full (with some delay),
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
failures with non-default export can be examined and handled accordingly in one run with the cache update,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
“interesting failures” (like the nested-language+polyglossia problem in es/Customization can be separated and moved into dedicated test samples.
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
When making non-trivial changes to LyX's \SpecialChar LaTeX
|
||
export code (e.g.
|
||
touching the encoding code or package handling code that you expect will change the exported \SpecialChar LaTeX
|
||
in some way):
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
\paragraph_spacing single
|
||
Consider running all of the export tests before and after your change.
|
||
If there are differences,
|
||
please reconcile these (i.e.
|
||
fix the bug or fix the tests)
|
||
\emph on
|
||
before
|
||
\emph default
|
||
committing.
|
||
Ask for help if you're not sure what to.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
If you do not want to run the tests,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
post the patch on the list and others will run the tests and eventually ask for fixes,
|
||
or
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
commit,
|
||
but be prepared to fix eventually arising problems or to revert the commit if there is no easy fix.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Understand how to interpret test failures.
|
||
If your commit is found to have broken a test,
|
||
you should be able to interpret the test results when made aware of them.
|
||
See Section
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "subsec:Interpreting-export-tests"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Subsubsection
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "par:export-test-output-formats"
|
||
|
||
\end_inset
|
||
|
||
Output formats
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The following output formats are currently tested for each sample document (see
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "par:Export-test-filtering"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
for exceptions):
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
LyX:
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
lyx16 LyX 1.6 file format (lyx2lyx)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
lyx21 LyX 2.1 file format (lyx2lyx)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
xhtml LyXHTML (native LyX HTML export)
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
LyX
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
+
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
LaTeX:
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
dvi DVI (8-bit latex)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
dvi3_texF DVI (LuaTeX with 8-bit TeX fonts)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
dvi3_systemF DVI (LuaTeX with Unicode fonts)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
pdf2 PDF (pdflatex)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
pdf4_texF PDF (XeTeX with 8-bit TeX fonts)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
pdf4_systemF PDF (XeTeX with Unicode fonts)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
pdf5_texF PDF (LuaTeX with 8-bit TeX fonts)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
pdf5_systemF PDF (LuaTeX with Unicode fonts)
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
LyX
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
+
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
LaTeX
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
+
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
postprocessing:
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
pdf DVI -> PS (dvips) -> PDF (ps2pdf)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
pdf3 DVI -> PDF (dvipdfm)
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
not
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
tested:
|
||
(or only if set as default output format in the document source)
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
latex LaTeX (plain)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
luatex LaTeX (LuaTeX)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
dviluatex LaTeX (dviluatex)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
pdflatex LaTeX (pdflatex)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
platex LaTeX (pLaTeX)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
xetex LaTeX (XeTeX)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
eps3 EPS (encapsulated Postscript) (cropped)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
ps DVI -> Postscript (dvips)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
odf
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
text (nor text2,
|
||
...,
|
||
text4)
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
textparagraph
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
word
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
word2
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring pdf5msystemFM
|
||
wordhtml
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Subsubsection
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "par:Configuring-ctests"
|
||
|
||
\end_inset
|
||
|
||
Configuring the tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
To enable the export autotests,
|
||
add the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-DLYX_ENABLE_EXPORT_TESTS=ON
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
flag.
|
||
For example:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
cmake -DLYX_ENABLE_EXPORT_TESTS=ON /path/to/source
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\noindent
|
||
This flag will increase the time for the cmake command by several seconds,
|
||
mainly because of the process of inverting tests (see Section
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "subsec:Interpreting-export-tests"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
).
|
||
\end_layout
|
||
|
||
\begin_layout Subsubsection
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "par:ctest-options"
|
||
|
||
\end_inset
|
||
|
||
Running the tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
To run all tests,
|
||
in the build directory simply run the command
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
A full,
|
||
up-to-date TeXLive installation is recommended to run the tests.
|
||
Otherwise,
|
||
some tests will fail.
|
||
Tests with additional requirements are labeled
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
unreliable:nonstandard
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
To run only some of the tests,
|
||
use command line options (see examples below):
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring -R
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-R <pattern>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
Run only the tests whose names match the given regular expression.
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring -R
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-L <pattern>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
Run only the tests whose labels match the given regular expression.
|
||
A test may have more that one label.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring -R
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-E <pattern>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
Exclude the tests whose names match the given regular expression.
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring -R
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-LE <pattern>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
Exclude the tests whose labels match the given regular expression.
|
||
Cannot be combined with
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-L
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The following options help to find good selection patterns:
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring -R
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-N
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
List the tests that would be run but not actually run them.
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
Useful in conjunction with the -R,
|
||
-L,
|
||
-E and -LE options,
|
||
e.g.,
|
||
if you want to know how many tests there are or whether your
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
<pattern>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
regular expression did what you expected.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring -R
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
\SpecialChar nobreakdash
|
||
\SpecialChar nobreakdash
|
||
print-labels
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
print the list of all labels associated with the test set.
|
||
Can also be combined with -R,
|
||
-L,
|
||
-E,
|
||
...
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Other useful options are:
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring -R
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-j <jobs>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
Run the tests in parallel using the given number of jobs.
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
We are still working on getting the tests to run in parallel.
|
||
However,
|
||
when running the tests in parallel,
|
||
sometimes tests fail that pass when run sequentially.
|
||
A reasonable approach is to first run the tests in parallel and then run the failed tests sequentially.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
For example,
|
||
to run 8 jobs at a time:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -j8
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest \SpecialChar nobreakdash
|
||
\SpecialChar nobreakdash
|
||
rerun-failed
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
When specifying a subset of the tests (e.g.
|
||
using
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
\SpecialChar nobreakdash
|
||
R <pattern>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
),
|
||
the same subset must be specified when using the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
\SpecialChar nobreakdash
|
||
\SpecialChar nobreakdash
|
||
rerun-failed
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
option because it is the test numbers that are used to index which tests failed on the previous run.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\noindent
|
||
Note that some tests cannot be run in parallel.
|
||
These tests are marked in the code with the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
RUN_SERIAL ON
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
CMake property.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring -R
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
\SpecialChar nobreakdash
|
||
\SpecialChar nobreakdash
|
||
timeout <seconds>
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
Set a global timeout on all tests that do not already have a timeout set on them.
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
There have been bugs in LyX and in \SpecialChar LaTeX
|
||
which cause compilation to hang,
|
||
and without a timeout a test might never stop (in one case there was even a memory leak).
|
||
If a test times out,
|
||
the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
command exits with error,
|
||
but you can distinguish between a timed out test and a failed test in the output reported at the end of the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
command.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Standard
|
||
See the manual (
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
man ctest
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
) the full list of command line options.
|
||
\end_layout
|
||
|
||
\begin_layout Subsubsection
|
||
Examples
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
run only the export tests:
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -L export
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
run inverted tests:
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -L "inverted|suspended"
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
list all export tests which match any of the labelling patterns:
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -N -R "
|
||
\backslash
|
||
..*_export/"
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
exclude rarely used output formats and post-processing tests
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -L export -E "_(texF|dvi3|pdf3?)"
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Subsubsection
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "subsec:Interpreting-export-tests"
|
||
|
||
\end_inset
|
||
|
||
Interpreting the export test results
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
A test can fail for several reasons,
|
||
not all of them bad.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
A new or edited sample document may be incompatible with some output formats.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
A dependency is not met (e.g.
|
||
the \SpecialChar LaTeX
|
||
class file).
|
||
One hint that this is the case is that the corresponding
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
check_load
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
test will likely also fail.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
An inverted test fails to fail (i.e.
|
||
export that previously failed now works).
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
An external dependency was updated (e.g.
|
||
\SpecialChar TeX
|
||
Live).
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
A recent code change introduced a bug.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "enu:exposed"
|
||
|
||
\end_inset
|
||
|
||
A change in a document exposed a previously unknown bug or an incompatibility with an export format (e.g.
|
||
Lua\SpecialChar LaTeX
|
||
).
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Because the .lyx files are exported in several formats,
|
||
it is not surprising that many of the exports fail.
|
||
This expectation of failure is addressed by
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
inverting
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
the tests,
|
||
that is,
|
||
by marking the test as
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
passing
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
if the export exits with error and as
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
failing
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
if the export succeeds
|
||
\emph on
|
||
.
|
||
|
||
\emph default
|
||
It follows that these expected failures will not show up as failed tests in the test results and thus will not pollute the
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
good
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
tests.
|
||
If the export actually succeeds,
|
||
then the test will fail.
|
||
The purpose of this failure is to get your attention—
|
||
something has changed,
|
||
possibly for the better.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
We try to document why a test is inverted or ignored.
|
||
See the comment (prefixed with
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
#
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
) above the block in which the test is listed as inverted or ignored in the files
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/autotests/invertedTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
,
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/autotests/unreliableTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
and
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/autotests/ignoredTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
A good question is why do we enable the tests for non-default formats?
|
||
The answer is that if a non-default route is broken it is often because a bug was introduced in LyX and not because a document-specific change was made that is not supported by the route.
|
||
In other words,
|
||
there is a high signal/noise ratio in the export tests for some non-default formats.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
When a test or several tests fail,
|
||
consider checking the files in the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
Testing/Temporary/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subdirectory of your build directory.
|
||
In this subdirectory are three files:
|
||
the file
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
LastTestsFailed.log
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
simply lists the tests that failed on your last
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
command;
|
||
the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
LastTest.log
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
file contains the output from the tests (and often has details explaining why a test failed);
|
||
and the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
CTestCostData.txt
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
file lists the times that it took to run the tests.
|
||
\end_layout
|
||
|
||
\begin_layout Subsubsection
|
||
What action should you take if a test fails?
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\paragraph_spacing single
|
||
It is always good to check manually why something fails and if it passes if the PDF output is good.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Generally,
|
||
if a change breaks compilation for the target format (for the manuals pdf2) without solving some important other issue,
|
||
|
||
\emph on
|
||
fix or revert the commit
|
||
\emph default
|
||
that led to failure.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
If it is not possible to (immediately) fix the failure but there are reasons not to revert the commit (e.g.
|
||
it fixes another more important issue),
|
||
|
||
\emph on
|
||
invert
|
||
\emph default
|
||
the failing test case (see
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "par:Inverted-tests"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
).
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
If an
|
||
\emph on
|
||
inverted
|
||
\emph default
|
||
test case fails because the export now works,
|
||
first confirm that the output of the corresponding export looks good (e.g.,
|
||
not garbled text).
|
||
Then,
|
||
|
||
\emph on
|
||
uninvert
|
||
\emph default
|
||
the test by removing the pattern from the
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
invertedTests
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
file (see
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "par:Inverted-tests"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
).
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
If the export did not fail previously but led to wrong output (PDF,
|
||
say),
|
||
\begin_inset Foot
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
Non-failing test with wrong output should be labeled as
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
unreliable:wrong_output
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
(
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "par:Unreliable-tests"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
).
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
it is in fact an improvement when the test now fails.
|
||
|
||
\emph on
|
||
Invert
|
||
\emph default
|
||
the failing test case (see
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "par:Inverted-tests"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
).
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
In case of tests failing due to missing requirements (tests labeled
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
unreliable:nonstandard
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
or testing on a system with only a subset of TeXLive installed),
|
||
ignore the failure,
|
||
ask for someone else to run the test,
|
||
or install the missing resources and try again.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Check the log file Testing/Temporary/LastTest.log.
|
||
In case of latex-errors rerun the failing test with environment variable 'LYX_DEBUG_LATEX' set to '1'.
|
||
This will include latex messages in LastTest.log,
|
||
so it should be easier to interpret the fail-reason.
|
||
\end_layout
|
||
|
||
\begin_layout Subsubsection
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "par:Inverted-tests"
|
||
|
||
\end_inset
|
||
|
||
Inverted tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Test cases whose name matches a pattern in the file
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/autotests/invertedTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
get the label
|
||
\emph on
|
||
inverted
|
||
\emph default
|
||
.
|
||
They get also the test property
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
WILL_FAIL
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
,
|
||
i.e.
|
||
they are reported as failing if the export works without error
|
||
\begin_inset Flex URL
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
https://cmake.org/cmake/help/v3.0/command/set_tests_properties.html
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Add failing cases to this file,
|
||
if they cannot be solved
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
immediately
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
but it is expected that the export will work in a foreseeable future,
|
||
e.g.
|
||
low priority issues like failures to export to a non-target format (for the manuals everything except pdf2).
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The following sublabels are currently present in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
invertedTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
:
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
todo test failures that require attention:
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
minor issues to explore and properly sort later,
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
easyfix issues,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
LyX bugs to report at trac (move pattern to section "lyxbugs" once done).
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Description
|
||
lyxbugs LyX bugs with a Trac number.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
ert Export failures due to "raw" LaTeX use in ERT or preamble code.
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
"Wontfix" if demonstrating correct use and OK in the default output format.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Description
|
||
texissues Export fails due to LaTeX limitations like non-ASCII characters in verbatim or listings,
|
||
incompatible packages,
|
||
...
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
"Wontfix" if documents demonstrate correct use in the default output format:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
If the source can be made more robust without becoming "hackish",
|
||
fix the source,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
if LyX could be enhanced to care for a permanent TeX limitation,
|
||
file a ticket at trac and add a pattern under lyxbugs,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
otherwise,
|
||
add a pattern here.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Description
|
||
attic Documents in the attic (kept for reference and format conversion test).
|
||
Usually
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
Wontfix
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Paragraph
|
||
Suspended tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Test cases whose name additionally matches a pattern in the file
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/autotests/suspendedTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
get the label
|
||
\emph on
|
||
suspended
|
||
\emph default
|
||
(instead of
|
||
\emph on
|
||
export
|
||
\emph default
|
||
and
|
||
\emph on
|
||
inverted
|
||
\emph default
|
||
).
|
||
This means they are not executed using
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -L export
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
or
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -L inverted
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
However,
|
||
they also get the test property
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
WILL_FAIL
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
,
|
||
i.e.
|
||
they are reported as failing if the export works without error.
|
||
From time to time they still have to be checked using
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -L suspended
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
These tests are suspended,
|
||
because the export fails for known reasons which cannot ATM be resolved.
|
||
But it is expected the reason might disappear in the future.
|
||
Be it new TL or better handling in \SpecialChar LyX
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
For ctest commands without the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-L
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
parameter nothing changes.
|
||
Suspended or not,
|
||
tests will be executed depending only on the selecting regular expression given to the ctest command (see
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "par:ctest-options"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
).
|
||
\end_layout
|
||
|
||
\begin_layout Subsubsection
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "par:Unreliable-tests"
|
||
|
||
\end_inset
|
||
|
||
Unreliable tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Test cases whose name matches a pattern in the file
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/autotests/unreliableTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
get the label
|
||
\emph on
|
||
unreliable
|
||
\emph default
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
These tests are not executed using
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -L export
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
or
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
ctest -L inverted
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
They pass or fail for various reasons not related to LyX (nonstandard,
|
||
erratic) or pass but should rather fail (wrong output).
|
||
|
||
\begin_inset Note Note
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
*invalid* tests (wrong output) are not *unreliable*.
|
||
# Use "unfit" or "unapplicable" as better label and name of pattern file?
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The following sublabels are currently present in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
unreliableTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
:
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
nonstandard Documents with additional requirements,
|
||
e.g.
|
||
a class or package file not in TeXLive.
|
||
|
||
\begin_inset Note Note
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
TODO:
|
||
rename to
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
extra
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
or
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
exotic
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
?
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
erratic Tests depending on local configuration or the phase of the moon.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
varying_versions Tests depending on e.g.
|
||
OS or version of a non-TeX-Live dependency.
|
||
Note that a full,
|
||
up-to-date TeX Live installation is required so this sublabel is about versions of other dependencies.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
wrong
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
output Export does not fail but the resulting document has (undetected) errors.
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
\paragraph_spacing single
|
||
\begin_inset Note Note
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
\paragraph_spacing single
|
||
These tests are in a strict sense not unreliable but
|
||
\emph on
|
||
invalid
|
||
\emph default
|
||
(not measuring what they should measure).
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Subsubsection
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "par:Export-test-filtering"
|
||
|
||
\end_inset
|
||
|
||
Export test filtering
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The assignment of a label to a test is controlled by a set of files with regular expressions that are matched against the test names.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
ignoredTests (small file)
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
Tests selected here are withdrawn in the configuration step (cf.
|
||
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "par:Configuring-ctests"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
).
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
Input Test of any export combination
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
Output Stop if tests not selected here
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Description
|
||
unreliableTests:
|
||
Tests selected pass or fail dependent on the system where the test is run.
|
||
Selected tests gain the label 'unreliable'.
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
Input Each test which passed 'ignoredTests'
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
Output Gain label 'unreliable',
|
||
proceed with checking for 'inverted'.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Description
|
||
invertedTests
|
||
\begin_inset space \space{}
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
Input Each test which passed 'ignoredTests'
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
Output Stop if not selected,
|
||
gain test-property 'WILL_FAIL' (i.e.
|
||
tests are reported as failing if the export works without error.) If no subselection applies,
|
||
gain labels 'export' and 'inverted'.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The following filter perfoms a subselection of 'invertedTests':
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
suspendedTests Tests selected here gain the label 'suspended' but _not_ 'export' or 'inverted',
|
||
although in ctest they remain inverted.
|
||
('ctest' knows only 'inverted' or not,
|
||
labels are used only for test selection)
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
Input Each test selected by 'invertedTests'
|
||
\end_layout
|
||
|
||
\begin_layout Labeling
|
||
\labelwidthstring 00.00.0000
|
||
Output Selected test gains label 'suspended'.
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\end_deeper
|
||
\begin_layout Standard
|
||
The following table may clarify label assignement
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset space \hspace{}
|
||
\length -3cm
|
||
\end_inset
|
||
|
||
|
||
\begin_inset Tabular
|
||
<lyxtabular version="3" rows="6" columns="8">
|
||
<features tabularvalignment="middle">
|
||
<column alignment="left" valignment="top" width="2cm">
|
||
<column alignment="left" valignment="top" width="2.5cm">
|
||
<column alignment="left" valignment="top" width="2cm">
|
||
<column alignment="center" valignment="top" width="2.5cm">
|
||
<column alignment="center" valignment="top">
|
||
<column alignment="center" valignment="top">
|
||
<column alignment="center" valignment="top">
|
||
<column alignment="center" valignment="top">
|
||
<row>
|
||
<cell multicolumn="1" alignment="left" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Test matching pattern in file:
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="2" alignment="left" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="2" alignment="left" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="2" alignment="left" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="1" alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Assigned label
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="2" alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="2" alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
test property
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
ignored\SpecialChar softhyphen
|
||
Tests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
unreliable\SpecialChar softhyphen
|
||
Tests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
inverted\SpecialChar softhyphen
|
||
Tests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
suspended\SpecialChar softhyphen
|
||
Tests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
export
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
inverted
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
suspended
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Yes
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell multirow="3" alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
No
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multirow="3" alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
If Yes,
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
add label
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
'unreliable'
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multirow="3" alignment="left" valignment="middle" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Yes
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Yes
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
+
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
WILL_FAIL
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell multirow="4" alignment="left" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multirow="4" alignment="left" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multirow="4" alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
No
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
+
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
+
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
WILL_FAIL
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell multirow="4" alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multirow="4" alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
No
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
+
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
</lyxtabular>
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset Note Note
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
Without the
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
suspendedTests
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
filter,
|
||
this would be far less complicated:
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
\begin_inset Tabular
|
||
<lyxtabular version="3" rows="6" columns="7">
|
||
<features tabularvalignment="middle">
|
||
<column alignment="left" valignment="top" width="0pt">
|
||
<column alignment="left" valignment="top" width="0pt">
|
||
<column alignment="left" valignment="top" width="0pt">
|
||
<column alignment="center" valignment="top">
|
||
<column alignment="center" valignment="top">
|
||
<column alignment="center" valignment="top">
|
||
<column alignment="center" valignment="top">
|
||
<row>
|
||
<cell multicolumn="1" alignment="left" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Test matching pattern in file:
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="2" alignment="left" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="2" alignment="left" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="1" alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Label
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="2" alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multicolumn="2" alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
test property
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
ignoredTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
unreliableTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
invertedTests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
export
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
unreliable
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
inverted
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Yes
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell multirow="3" alignment="left" valignment="middle" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
No
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Yes
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
+
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell multirow="4" alignment="left" valignment="middle" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multirow="3" alignment="left" valignment="middle" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
No
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
Yes
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
+
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
+
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
WILL_FAIL
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
<row>
|
||
<cell multirow="4" alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell multirow="4" alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="left" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
No
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" leftline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
+
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
-
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
<cell alignment="center" valignment="top" topline="true" bottomline="true" rightline="true" usebox="none">
|
||
\begin_inset Text
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
</cell>
|
||
</row>
|
||
</lyxtabular>
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Subsection
|
||
check_load tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
These tests check whether a .lyx file loads without any terminal messages.
|
||
They correspond to the manual operations of simply opening a .lyx file on the terminal,
|
||
exiting LyX once the file is loaded,
|
||
and then checking whether there is any output from the terminal.
|
||
These tests are useful for catching malformed .lyx files and parsing bugs.
|
||
They can also be used to find a .lyx file in which an instance of something happens.
|
||
To do this,
|
||
compile LyX with a local patch that outputs something to the terminal when an instance is found,
|
||
and then run the check_load tests to see if any fail,
|
||
which would mean that the situation occurs in the LyX documentation files corresponding to the failed tests.
|
||
These tests are expectedly fragile:
|
||
any LyX diagnostic message,
|
||
which is not necessarily an error,
|
||
would cause the tests to fail.
|
||
Similarly,
|
||
any message output by a library (e.g.
|
||
Qt) would also cause failure.
|
||
There are some messages that the check_load tests are instructed to ignore,
|
||
which are stored in the file
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/autotests/filterCheckWarnings
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Under cmake,
|
||
the tests are labeled as 'load'.
|
||
\end_layout
|
||
|
||
\begin_layout Subsection
|
||
Keytests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Automated tests based on the "MonKey Testing" keytest program are enabled if the necessary dependencies are found and if the CMake flag
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-DLYX_ENABLE_KEYTESTS=ON
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
is used.
|
||
They are documented in the README document in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
development/autotests
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
subfolder of the \SpecialChar LyX
|
||
source code distribution.
|
||
\end_layout
|
||
|
||
\begin_layout Subsection
|
||
lyx21 tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
These tests combine lyx2lyx tests with check_load tests.
|
||
They fail if either fails.
|
||
\end_layout
|
||
|
||
\begin_layout Subsection
|
||
URL tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The URL tests are enabled with the
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
-DLYX_ENABLE_URLTESTS=ON
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
CMake flag and are useful for finding broken links in our documentation files.
|
||
If a URL test fails,
|
||
to see which link in particular was reported as broken,
|
||
see the output in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
LastTest.log
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
These tests are extremely fragile (e.g.
|
||
a test can depend on your Internet connection) and a failed URL test should not be taken too seriously.
|
||
URL tests are labeled as
|
||
\family typewriter
|
||
'url'.
|
||
\end_layout
|
||
|
||
\begin_layout Subsubsection
|
||
Running URL tests
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
cmake is required to run the \SpecialChar LyX
|
||
tests,
|
||
running them is not implemented for autotools.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The appropriate commands are:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
|
||
\family typewriter
|
||
ctest -L url
|
||
\family default
|
||
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
runs all tests with label
|
||
\family typewriter
|
||
'url'
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
|
||
\family typewriter
|
||
ctest -R 'check_.*urls'
|
||
\family default
|
||
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
runs the tests 'check_accessible_urls'
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Associated test results can be examined in ctest-log directory in files of the form 'LastFailed.*URLS.log'
|
||
\end_layout
|
||
|
||
\begin_layout Chapter
|
||
Development policies
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
This chapter lists some guidelines that should be followed.
|
||
This list is not complete,
|
||
and many guidelines are in separate chapters,
|
||
such as
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
When is an update of the .lyx file format number needed?
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
in Section
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "sec:When-is-an"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
When to set a fixed milestone?
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Only set a fixed milestone (like 2.1.4 or 2.2.0) if at least one of the following holds:
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Somebody is actively working on a fix.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
The bug is so severe that it would block the release if it is not fixed.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
If a bug is important,
|
||
but nobody is working on it,
|
||
and it is no showstopper,
|
||
use a milestone like 2.1.x or 2.2.x.
|
||
For all other bugs,
|
||
do not set a milestone at all.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Can we add rc entries in stable branch?
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
No.
|
||
We are supposed to increase the prefs2prefs version number with such things.
|
||
\end_layout
|
||
|
||
\begin_layout Chapter
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "sec:Documentation-policies"
|
||
|
||
\end_inset
|
||
|
||
Documentation policies
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Rules
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
There are 6
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
rules in editing the docs:
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "enu:If-you-are"
|
||
|
||
\end_inset
|
||
|
||
If you are not the maintainer of a doc file or a chapter/section,
|
||
you MUST use change tracking so that the maintainer could review your changes
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Respect the formatting of the document.
|
||
The different files use different formatting styles.
|
||
That is OK and has historic reasons nobody fully knows ;-).
|
||
But it is important to be consistent within one file.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
All changes you make to a file in one language MUST also go the file in the other actively maintained languages.
|
||
Normally the maintainer does this for you,
|
||
if you are the maintainer,
|
||
you must do this by copying or changing the changed or added text to the other files so that the translators sees the blue underlined text and know what they have to translate and what was changed.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
You MUST assure that the document is compilable as
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
PDF (pdflatex)
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
or the document's default output format after your changes.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
All fixes (typos,
|
||
compilation fixes,
|
||
updates info etc.) go at first into the current Git branch because the user should benefit from all fixes with every minor release.
|
||
Feel free to commit directly to branch as long as you follow rule
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "enu:If-you-are"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
You can immediately commit to master as well.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "enu:The-fileformat-of"
|
||
|
||
\end_inset
|
||
|
||
The fileformat of a file must not be changed unless you document a new feature in LyX that requires a new fileformat.
|
||
The reason for this rule is to keep it easy for the doc maintainers to port/backport changes to from master/branch.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The main documentation consists of these files:
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
Welcome.lyx It is the first file you see after an installation.
|
||
We assume that a new user sees this.
|
||
It is therefore designed to be as simple as possible.
|
||
Therefore please don't add any new formatting,
|
||
only fix typos etc.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
Intro.lyx This is the manual new users will read to learn \SpecialChar LyX
|
||
.
|
||
It therefore uses a limited set of formatting.
|
||
For example a standard document class.
|
||
Since new users will first learn about the formatting possibilities of \SpecialChar LyX
|
||
please keep this file that simple.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
Tutorial.lyx Our tutorial.
|
||
It must be always up to date.
|
||
Normally there is nothing to add since we don't want to overwhelm new users with too much details.
|
||
They will learn these details while using \SpecialChar LyX
|
||
and we have special manuals.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
UserGuide.lyx Our main user guide.
|
||
It covers a mixture of basic and detailed information.
|
||
Some information is also in the Math and EmbeddedObjects manual so that the UserGuide refers to these files.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
EmbeddedObjects.lyx A special manual to explain things like tables floats boxes etc.
|
||
in all detail.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
Math.lyx A special manual to explain everything regarding math in all detail.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
Additional.lyx This manual covers information that would be too much detail for the UserGuide or would make the UserGuide uncompilable or only compilable when installing a lot of special \SpecialChar LaTeX
|
||
packages.
|
||
What should be in the UserGuide or better in Additional is a matter of taste.
|
||
It is up to you to decide that.
|
||
Additional.lyx is not completely up to date (only chapter
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
8 is up to date).
|
||
It certainly needs a rewrite and update.
|
||
For example many info in chapter
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
2 and 3 are already covered by the UserGuide and/or the EmbeddedObjects manual.
|
||
\end_layout
|
||
|
||
\begin_layout Description
|
||
Customization.lyx This manual covers information how to customize \SpecialChar LyX
|
||
for certain output formats,
|
||
operating systems,
|
||
languages etc.
|
||
It is currently completely out of date and needs a major rewrite and update.
|
||
If you do this please assure that your information are given for all OSes and \SpecialChar LaTeX
|
||
distributions (meaning be as objective as possible).
|
||
\end_layout
|
||
|
||
\begin_layout Chapter
|
||
Coding rules
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The aim of this chapter is to serve as a guide for the developers,
|
||
to aid us to get clean and uniform code.
|
||
It is incomplete.
|
||
We really like to have new developers joining the \SpecialChar LyX
|
||
Project.
|
||
However,
|
||
we have had problems in the past with developers leaving the project and their contributed code in a far from perfect state.
|
||
Most of this happened before we really became aware of these issues,
|
||
but still,
|
||
we don't want it to happen again.
|
||
So we have put together some guidelines and rules for the developers.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
General
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
These guidelines should save us a lot of work while cleaning up the code and help us to have quality code.
|
||
\SpecialChar LyX
|
||
has been haunted by problems coming from unfinished projects by people who have left the team.
|
||
Those problems will hopefully disappear if the code is easy to hand over to somebody else.
|
||
In general,
|
||
if you want to contribute to the main source,
|
||
we expect at least that you:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
The most important rule first:
|
||
KISS (Keep It Simple Stupid),
|
||
always use a simple implementation in favor of a more complicated one.
|
||
This eases maintenance a lot.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Write good C++ code:
|
||
readable,
|
||
well commented,
|
||
and taking advantage of the OO model.
|
||
Follow the formatting guidelines.
|
||
See sec.
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "sec:Formatting"
|
||
plural "false"
|
||
caps "false"
|
||
noprefix "false"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
As of LyX 2.4.0,
|
||
you can use features of C++11.
|
||
Accordingly you have to use C++11 standard conforming compiler,
|
||
e.
|
||
\begin_inset space \thinspace{}
|
||
\end_inset
|
||
|
||
g.
|
||
not too dated version of GCC or Clang.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Adapt the code to the structures already existing in \SpecialChar LyX
|
||
,
|
||
or in the case that you have better ideas,
|
||
discuss them on the developer's list before writing the code.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Take advantage of the C++ standard library.
|
||
Especially don't use custom containers when a standard container is usable;
|
||
learn to use the algorithms and functors in the standard library.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Be aware of exceptions and write exception safe code.
|
||
See sec.
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "sec:Exceptions"
|
||
plural "false"
|
||
caps "false"
|
||
noprefix "false"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Document all variables,
|
||
methods,
|
||
functions,
|
||
classes etc.
|
||
We are using the source documentation program doxygen,
|
||
a program that handles javadoc syntax,
|
||
to document sources.
|
||
You can download doxygen from:
|
||
|
||
\begin_inset Flex URL
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
http://www.stack.nl/~dimitri/doxygen/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
We have certain code constructs that we try to follow.
|
||
See sec.
|
||
\begin_inset space ~
|
||
\end_inset
|
||
|
||
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "sec:Code-constructs"
|
||
plural "false"
|
||
caps "false"
|
||
noprefix "false"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Submitting code
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
It is implicitly understood that all patches contributed to The \SpecialChar LyX
|
||
Project is under the Gnu General Public License,
|
||
version 2 or later.
|
||
If you have a problem with that,
|
||
don't contribute code.
|
||
Also please don't just pop up out of the blue with a huge patch (or small) that changes something substantial in \SpecialChar LyX
|
||
.
|
||
Always discuss your ideas with the developers on the developer's mailing list.
|
||
When you create the patch,
|
||
please use
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
|
||
\family typewriter
|
||
diff -up
|
||
\family default
|
||
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
since we find that a lot easier to read than the other diff formats.
|
||
Also please do not send patches that implements or fixes several different things;
|
||
several patches is a much better option.
|
||
We also require you to provide a commit message entry with every patch,
|
||
this describes in detail what the patch is doing.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Code constructs
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "sec:Code-constructs"
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
We have several guidelines on code constructs,
|
||
some of these exist to make the code faster,
|
||
others to make the code clearer.
|
||
Yet others exist to allow us to take advantage of the strong type checking in C++.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Declaration of variables should wait as long as possible.
|
||
The rule is:
|
||
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
Don't declare it until you need it.
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
In C++ there are a lot of user defined types,
|
||
and these can very often be expensive to initialize.
|
||
This rule connects to the next rule too.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Declare the variable as
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
const
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
if you don't need to change it.
|
||
This applies to POD types like
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
int
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
as well as classes.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Make the scope of a variable as small as possible.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Make good use of namespaces.
|
||
Prefer anonymous namespaces to declaring
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
static
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
for file scope.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Prefer preincrement to postincrement whenever possible.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Preincrement has potential of being faster than postincrement.
|
||
Just think about the obvious implementations of pre/post-increment.
|
||
This rule applies to decrement too.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Use:
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
++T;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
--U;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Do not use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
T++;
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
U--;
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Try to minimize evaluation of the same code over and over.
|
||
This is aimed especially at loops.
|
||
|
||
\begin_inset Newline newline
|
||
\end_inset
|
||
|
||
Use:
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
Container::iterator end = large.end();
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
for (Container::iterator it = large.begin();
|
||
it != end;
|
||
++it) {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
...;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
}
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Or better (C++11):
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
for (auto const & it :
|
||
large) {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
...;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
}
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Do not use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
for (Container::iterator it = large.begin();
|
||
it != large.end();
|
||
++it) {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
...;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
}
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
For functions and methods that return a non-POD type
|
||
\begin_inset Foot
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
Plain Ol' Data type
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
T
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
,
|
||
return
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
T const
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
instead.
|
||
This gives better type checking,
|
||
and will give a compiler warning when temporaries are used wrongly.
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
Use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
T const add(...);
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Do not use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
T add(...);
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Avoid using the default cases in switch statements unless you have too.
|
||
Use the correct type for the switch expression and let the compiler ensure that all cases are exhausted.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
enum Foo {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
FOO_BAR1,
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
FOO_BAR2
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
};
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
Foo f = ...;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
switch (f) {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
case FOO_BAR1:
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
...;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
break;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
case FOO_BAR2:
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
...;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
break;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
default:
|
||
// not needed and would shadow a wrong use of Foo
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
...;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
break;
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
}
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Use default initialization such as
|
||
\begin_inset listings
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
int i = 0;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
Class * ptr = nullptr;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
rather than brace initialization:
|
||
\begin_inset listings
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
int i {};
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
Class * ptr {};
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
for PODs.
|
||
Use brace initialization only for more complex data structures.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Exceptions
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "sec:Exceptions"
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Be aware of the presence of exceptions.
|
||
One important thing to realize is that you often do not have to use throw,
|
||
try or catch to be exception safe.
|
||
Let's look at the different types of exceptions safety (these are taken from Herb Sutter's book
|
||
\begin_inset CommandInset citation
|
||
LatexCommand cite
|
||
key "sutter"
|
||
literal "false"
|
||
|
||
\end_inset
|
||
|
||
):
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Basic guarantee:
|
||
Even in the presence of exceptions thrown by T or other exceptions,
|
||
Stack objects don't leak resources.
|
||
Note that this also implies that the container will be destructible and usable even if an exception is thrown while performing some container operation.
|
||
However,
|
||
if an exception is thrown,
|
||
the container will be in a consistent,
|
||
but not necessarily predictable,
|
||
state.
|
||
Containers that support the basic guarantee can work safely in some settings.
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Strong guarantee:
|
||
If an operation terminates because of an exception,
|
||
program state will remain unchanged.
|
||
This always implies commit-or-rollback semantics,
|
||
including that no references or iterators into the container be invalidated if an operation fails.
|
||
For example,
|
||
if a Stack client calls Top and then attempts a Push that fails because of an exception,
|
||
then the state of the Stack object must be unchanged and the reference returned from the prior call to Top must still be valid.
|
||
For more information on these guarantees,
|
||
see Dave Abrahams's documentation of the SGI exception-safe standard library adaption at:
|
||
|
||
\begin_inset Flex URL
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
http://www.stlport.org/doc/exception_safety.html
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
Probably the most interesting point here is that when you implement the basic guarantee,
|
||
the strong guarantee often comes for free.
|
||
For example,
|
||
in our Stack implementation,
|
||
almost everything we did was needed to satisfy just the basic guarantee – and what's presented above very nearly satisfies the strong guarantee,
|
||
with little or no extra work.
|
||
Not half bad,
|
||
considering all the trouble we went to.
|
||
In addition to these two guarantees,
|
||
there is one more guarantee that certain functions must provide in order to make overall exception safety possible:
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
No throw guarantee:
|
||
The function will not emit an exception under any circumstances.
|
||
Overall exception safety isn't possible unless certain functions are guaranteed not to throw.
|
||
In particular,
|
||
we've seen that this is true for destructors;
|
||
later in this miniseries,
|
||
we'll see that it's also needed in certain helper functions,
|
||
such as
|
||
\family typewriter
|
||
Swap()
|
||
\family default
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
For all cases where we might be able to write exception safe functions without using try,
|
||
throw or catch we should do so.
|
||
In particular we should look over all destructors to ensure that they are as exception safe as possible.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
Formatting
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "sec:Formatting"
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Only one declaration on each line.
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
Use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
int a;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
int b;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Do not use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
int a,
|
||
b;
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
This is especially important when initialization is done at the same time:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
string a = "Lars";
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
string b = "Gullik";
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Do not use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
string a = "Lars",
|
||
b = "Gullik";
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
[Note that
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
string a = "Lars"
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
is formally calling a copy constructor on a temporary constructed from a string literal and therefore has the potential of being more expensive then direct construction by
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
string a("Lars")
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
However the compiler is allowed to elide the copy (even if it had side effects),
|
||
and modern compilers typically do so.
|
||
Given these equal costs,
|
||
\SpecialChar LyX
|
||
code favours the '=' idiom as it is in line with the traditional C-style initialization,
|
||
|
||
\emph on
|
||
and
|
||
\emph default
|
||
cannot be mistaken as function declaration,
|
||
|
||
\emph on
|
||
and
|
||
\emph default
|
||
reduces the level of nested parentheses in more initializations.]
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Pointers and references:
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
Use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
char * p = "flop";
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
char & c = *p;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Do not use:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
char *p = "flop";
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
char &c = *p;
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Some time ago we had a huge discussion on this subject and after convincing argumentation from Asger this is what we decided.
|
||
Also note that we will have:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
char const * p;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
and not
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
const char * p;
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Operator names and parentheses
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
operator==(type)
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
and not
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
operator == (type) // not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The == is part of the function name,
|
||
separating it makes the declaration look like an expression.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Function names and parentheses
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
void mangle()
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
and not
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
void mangle () // not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Enumerators
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
enum Foo {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
FOO_ONE = 1,
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
FOO_TWO = 2,
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
FOO_THREE = 3
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
};
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
or (C++11)
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
enum class Foo {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
One = 1,
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
Two = 2,
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
Three = 3
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
};
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
and not
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
enum { one = 1,
|
||
two = 2,
|
||
three 3 };
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
and not
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
enum {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
One = 1,
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
Two = 2,
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
Three = 3
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
};
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Null pointers
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
Use nullptr (C++11):
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
void * p = nullptr;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
and not
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
void * p = NULL;
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
and not
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
void * p = '
|
||
\backslash
|
||
0';
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
and not
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
void * p = 42 - 7 * 6;
|
||
// not used in LyX
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Note:
|
||
As an exception,
|
||
imported third party code as well as code interfacing the
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
native
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
APIs (
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
src/support/os_*
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
) can use NULL.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Naming rules for classes
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
Use descriptive but simple and short names.
|
||
Do not abbreviate.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Class names are usually capitalized,
|
||
and function names lowercased.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Enums are named like Classes,
|
||
values are usually in lower-case.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Public API functions are camel-case (
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
void setAFlagToAValue(bool)
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
).
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Member variables are underscored (
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
enable_this_feature_flag_
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
) with a final
|
||
\begin_inset Quotes eld
|
||
\end_inset
|
||
|
||
_
|
||
\begin_inset Quotes erd
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Private/protected functions are also camel-case.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
New types are capitalized,
|
||
so this goes for typedefs,
|
||
classes,
|
||
structs and enums.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Formatting
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
Adapt the formatting of your code to the one used in the other parts of \SpecialChar LyX
|
||
.
|
||
In case there is different formatting for the same construct,
|
||
use the one used more often.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Use existing structures
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
\begin_inset CommandInset label
|
||
LatexCommand label
|
||
name "Use-string-wherever"
|
||
|
||
\end_inset
|
||
|
||
Use
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
string
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
wherever possible.
|
||
Unicode strings should prefer using
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
docstring
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
instead of UTF-8 encoded
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
std::string
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Check out the filename and path tools in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
filetools.h
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Check out the string tools in
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
lstring.h
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Use the \SpecialChar LyX
|
||
Err class to report errors and messages using the lyxerr instantiation.
|
||
[add description of other existing structures]
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Declarations
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
Use this order for the access sections of your class:
|
||
public,
|
||
protected,
|
||
private.
|
||
The public section is interesting for every user of the class.
|
||
The private section is only of interest for the implementors of the class (you).
|
||
[Obviously not true since this is for developers,
|
||
and we do not want one developer only to be able to read and understand the implementation of class internals.
|
||
Lgb]
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Avoid declaring global objects in the declaration file of the class.
|
||
If the same variable is used for all objects,
|
||
use a static member.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Avoid global or static variables.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
File headers
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
If you create a new file,
|
||
the top of the file should look something like this :
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
/**
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
*
|
||
\backslash
|
||
file NewFile.cpp
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
* This file is part of LyX,
|
||
the document processor.
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
* Licence details can be found in the file COPYING.
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
*
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
*
|
||
\backslash
|
||
author Kaiser Sose
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
*
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
* Full author contact details are available
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
* in file CREDITS.
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
*/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Documentation
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
The documentation is generated from the header files.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
You document for the other developers,
|
||
not for yourself.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
You should document what the function does,
|
||
not the implementation.
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
in the .cpp files you document the implementation.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
Single line description (
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
///
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
),
|
||
multiple lines description (
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
/** ...
|
||
*/
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
),
|
||
see the doxygen webpage referenced above.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Section
|
||
Naming rules for \SpecialChar LyX
|
||
User Functions (LFUNs)
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Here is the set of rules to apply when a new command name is introduced:
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Use the object.event order.
|
||
That is,
|
||
use `word-forward' instead of `forward-word'.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Don't introduce an alias for an already named object.
|
||
Same for events.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Forward movement or focus is called `forward' (not `right').
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Backward movement or focus is called `backward' (not `left').
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Upward movement of focus is called `up'.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
Downward movement is called `down'.
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
The begin of an object is called `begin' (not `start').
|
||
\end_layout
|
||
|
||
\begin_layout Enumerate
|
||
The end of an object is called `end'.
|
||
\end_layout
|
||
|
||
\begin_layout Section
|
||
How to create class interfaces
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
(a.k.a How Non-Member Functions Improve Encapsulation)
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
I recently read an article by Scott Meyers,
|
||
where he makes a strong case on how non-member functions makes classes more encapsulated,
|
||
not less.
|
||
Just skipping to the core of this provides us with the following algorithm for deciding what kind of function to add to a class interface:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
We need to add a function f to the class C's API.
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
if (f needs to be virtual)
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
make f a member function of C;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
else if (f is operator>> or operator<<) {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
make f a non-member function;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
if (f needs access to non-public members of C)
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
make f a friend of C;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
} else if (f needs type conversions on its left-most argument) {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
make f a non-member function;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
if (f needs access to non-public members of C)
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
make f a friend of C;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
} else if (f can be implemented via C's public interface)
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
make f a non-member function;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
else
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
make f a member function of C;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Chapter
|
||
Coding recommendations
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
These are some rules for effective C++ programming.
|
||
These are taken from Scott Meyers
|
||
\begin_inset CommandInset citation
|
||
LatexCommand cite
|
||
key "journal"
|
||
literal "true"
|
||
|
||
\end_inset
|
||
|
||
,
|
||
and are presented in their short form.
|
||
These are not all the rules Meyers presents,
|
||
only the most important of them.
|
||
\SpecialChar LyX
|
||
does not yet follow these rules,
|
||
but they should be the goal.
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
use
|
||
\family typewriter
|
||
const
|
||
\family default
|
||
and
|
||
\family typewriter
|
||
inline
|
||
\family default
|
||
instead of
|
||
\family typewriter
|
||
#define
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
use the same form in corresponding calls to new and delete,
|
||
i.e.
|
||
write
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
delete[] obj;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
if
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
new obj[n];
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
was used to create the object and write
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
delete obj;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
if you wrote
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
new obj;
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
Notice strings should be
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
std::string
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
's instead of
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
char *
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
's.
|
||
(this contradicts to
|
||
\begin_inset CommandInset ref
|
||
LatexCommand ref
|
||
reference "Use-string-wherever"
|
||
nolink "false"
|
||
|
||
\end_inset
|
||
|
||
)
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
define a default constructor,
|
||
copy constructor and an assignment operator for all classes with dynamically allocated memory that are not made noncopyable
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
do not define default constructor,
|
||
copy constructor and an assignment operator if the compiler generated one would do the same
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
make destructors virtual in base classes and only there
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
assign to all data members in operator=()
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
strive for class interfaces that are complete and minimal
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
differentiate among member functions,
|
||
global functions and friend functions
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
avoid data members in the public interface
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
use const whenever possible
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
pass and return objects by reference instead of by value
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
choose carefully between function overloading and parameter defaulting
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
never return a reference to a local object or a dereferenced pointer initialized by new within the function
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
use enums for integral constants
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
minimize compilation dependencies between files
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
pay attention to compiler warnings
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
differentiate between inheritance of interface and inheritance of implementation
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
differentiate between inheritance and templates
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
ensure that global objects are initialized before they are used
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
avoid conditions to
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
if
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
and
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
while
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
that span more than a line
|
||
\end_layout
|
||
|
||
\begin_layout Chapter
|
||
\start_of_appendix
|
||
Notes
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
And one of mine:
|
||
(Lgb)
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
when switching on enums,
|
||
refrain from using "default:" if possible
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
And one of mine:
|
||
(Andre')
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
try to implement your class in a way that the automatically generated copy constructor and copy assignment work out-of-the box
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
I don't have problems with using boost in the implementation _if and only if_ it provides actual benefits over less intrusive alternatives.
|
||
I do have a problem with needlessly sprinkling 'boost::' over interfaces,
|
||
especially if it does not add any value.
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
Given that there seems to be an unconditional "typedef unsigned int quint32;" in qglobal.h I don't think there's any platform supported by current \SpecialChar LyX
|
||
that could not use 'unsigned int' (and an static assert in some implementation file for the unlikely case some ILP64 zombie raises its ugly head again.
|
||
And if that happens,
|
||
using <cstdint> would still be a better choice...)
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
The idea is to create something that's not compilable as soon as the condition is violated.
|
||
There are lots of possibilities to achieve this,
|
||
some examples follow:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
In C++11 there's a "built-in":
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
static_assert(sizeof(int) == 4,
|
||
"Funny platform")
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
until then on namespace scope:
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
#include <boost/static_assert.hpp>
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
BOOST_STATIC_ASSERT(sizeof(int) == 4)
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
or without boost:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
template<bool Condition>
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
struct static_assert_helper;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
template <>
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
struct static_assert_helper<true> {};
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
enum {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
dummy = sizeof(static_assert_helper<sizeof(int) == 4>)
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
};
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
or somewhat brutish without templates,
|
||
in any function:
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
\begin_inset listings
|
||
lstparams "basicstyle={\ttfamily},showstringspaces=false,tabsize=4"
|
||
inline false
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
const int d = sizeof(int) - 4;
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
switch (0) {
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
case 0:
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
case !(d*d):
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
break;
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
}
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_layout Standard
|
||
Any of them in a .cpp file will break compilation as soon as
|
||
\begin_inset Flex Code
|
||
status collapsed
|
||
|
||
\begin_layout Plain Layout
|
||
sizeof(int)
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
is not equal 4.
|
||
Personally I prefer something like the third version (or the first,
|
||
if using C++11 is allowed).
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
And one of mine:
|
||
(vfr)
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
On dynamics_casts
|
||
\begin_inset Flex URL
|
||
status open
|
||
|
||
\begin_layout Plain Layout
|
||
|
||
http://www.lyx.org/trac/changeset/35855
|
||
\end_layout
|
||
|
||
\end_inset
|
||
|
||
:
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Standard
|
||
A dynamic_cast is necessary when:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
the object to be casted is from an external library because we can't add Qxxx::asXxxx() to Qt e.g.:
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
QAbstractListModel to GuiIdListModel,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
QValidator to PathValidator,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
QWidget to TabWorkArea,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
QWidget to GuiWorkArea;
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
the object is to be casted from an interface to the implementing class,
|
||
because the Interface does not know by whom it is implemented:
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
ProgressInterface to GuiProgress,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
Application to GuiApplication.
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Standard
|
||
A dynamic_cast can be replaced by:
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
already existing as***Inset() functions,
|
||
e.g.:
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
asHullInset(),
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
asInsetMath()->asMacro(),
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
asInsetText();
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\begin_layout Itemize
|
||
A static_cast when we are sure this can't go wrong,
|
||
e.g.:
|
||
\begin_inset Separator latexpar
|
||
\end_inset
|
||
|
||
|
||
\end_layout
|
||
|
||
\begin_deeper
|
||
\begin_layout Itemize
|
||
we are sure that CellData::inset->clone() is an InsetTableCell,
|
||
\end_layout
|
||
|
||
\begin_layout Itemize
|
||
in cases where we explicitly check it->lyxCode().
|
||
\end_layout
|
||
|
||
\end_deeper
|
||
\end_deeper
|
||
\end_deeper
|
||
\begin_layout Bibliography
|
||
\begin_inset CommandInset bibitem
|
||
LatexCommand bibitem
|
||
key "meyers"
|
||
literal "true"
|
||
|
||
\end_inset
|
||
|
||
Meyers,
|
||
Scott.
|
||
Effective C++:
|
||
50 Specific Ways to Improve Your Programs and Design.
|
||
Addison-Wesley,
|
||
1992.
|
||
\end_layout
|
||
|
||
\begin_layout Bibliography
|
||
\begin_inset CommandInset bibitem
|
||
LatexCommand bibitem
|
||
key "sutter"
|
||
literal "true"
|
||
|
||
\end_inset
|
||
|
||
Sutter,
|
||
Herb.
|
||
Exceptional C++:
|
||
47 engineering puzzles,
|
||
programming problems,
|
||
and solutions.
|
||
ISBN 0-201-61562-2.
|
||
\end_layout
|
||
|
||
\begin_layout Bibliography
|
||
\begin_inset CommandInset bibitem
|
||
LatexCommand bibitem
|
||
key "journal"
|
||
literal "true"
|
||
|
||
\end_inset
|
||
|
||
Meyers,
|
||
Scott.
|
||
C/C++ User's Journal (Vol.18,
|
||
No.2).
|
||
\end_layout
|
||
|
||
\end_body
|
||
\end_document
|