diff mbox series

Document IntegerRange in internal documentation.

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

Commit Message

Martin Liška June 12, 2018, 2:24 p.m. UTC
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. UTC | #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. UTC | #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
diff mbox series

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