Document IntegerRange in internal documentation.

Message ID 80ec1149-bcf3-8629-910f-b86194d45bde@suse.cz
State New
Headers show
Series
  • Document IntegerRange in internal documentation.
Related show

Commit Message

Martin Liška June 12, 2018, 2:24 p.m.
Hi.

The patch is mentioning IntegerRange in documentation.

Ready for trunk?
Martin

gcc/ChangeLog:

2018-06-12  Martin Liska  <mliska@suse.cz>

	* doc/options.texi: Document IntegerRange.
---
 gcc/doc/options.texi | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Kyrill Tkachov June 12, 2018, 2:27 p.m. | #1
Hi Martin,

Just a couple of typos...

On 12/06/18 15:24, Martin Liška wrote:
> Hi.

>

> The patch is mentioning IntegerRange in documentation.

>

> Ready for trunk?

> Martin

>

> gcc/ChangeLog:

>

> 2018-06-12  Martin Liska  <mliska@suse.cz>

>

>         * doc/options.texi: Document IntegerRange.

> ---

>  gcc/doc/options.texi | 5 +++++

>  1 file changed, 5 insertions(+)

>

>

  
+@item IntegerRange(@var{n}, @var{m})
+The options's arguments are integer numbers.  The option parser

"option's"

+will check that value of an option is inclusively withing
+range [@var{n}, @var{m}].

"within the range ..."

Cheers,
Kyrill
Martin Liška June 12, 2018, 2:41 p.m. | #2
On 06/12/2018 04:27 PM, Kyrill Tkachov wrote:
> Hi Martin,

> 

> Just a couple of typos...

> 

> On 12/06/18 15:24, Martin Liška wrote:

>> Hi.

>>

>> The patch is mentioning IntegerRange in documentation.

>>

>> Ready for trunk?

>> Martin

>>

>> gcc/ChangeLog:

>>

>> 2018-06-12  Martin Liska  <mliska@suse.cz>

>>

>>         * doc/options.texi: Document IntegerRange.

>> ---

>>  gcc/doc/options.texi | 5 +++++

>>  1 file changed, 5 insertions(+)

>>

>>

>  

> +@item IntegerRange(@var{n}, @var{m})

> +The options's arguments are integer numbers.  The option parser

> 

> "option's"

> 

> +will check that value of an option is inclusively withing

> +range [@var{n}, @var{m}].

> 

> "within the range ..."

> 

> Cheers,

> Kyrill


Thanks for the review Kyrill!
Patch installed as r261498.

Martin

Martin

Patch

diff --git a/gcc/doc/options.texi b/gcc/doc/options.texi
index 69743125c22..f30f077983f 100644
--- a/gcc/doc/options.texi
+++ b/gcc/doc/options.texi
@@ -264,6 +264,11 @@  option handler.  @code{UInteger} should also be used on options like
 @code{-falign-loops}=@var{n} are supported to make sure the saved
 options are given a full integer.
 
+@item IntegerRange(@var{n}, @var{m})
+The options's arguments are integer numbers.  The option parser
+will check that value of an option is inclusively withing
+range [@var{n}, @var{m}].
+
 @item ToLower
 The option's argument should be converted to lowercase as part of
 putting it in canonical form, and before comparing with the strings