public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
From: Pierre-Marie de Rodat <pmderodat@gcc.gnu.org>
To: gcc-cvs@gcc.gnu.org
Subject: [gcc r13-1655] [Ada] Refine type for checking number of pragma arguments
Date: Wed, 13 Jul 2022 10:02:40 +0000 (GMT)	[thread overview]
Message-ID: <20220713100240.D33323851A9D@sourceware.org> (raw)

https://gcc.gnu.org/g:1d73dfb5e6f60030bb0809266fb9c639eb60fa11

commit r13-1655-g1d73dfb5e6f60030bb0809266fb9c639eb60fa11
Author: Piotr Trojanek <trojanek@adacore.com>
Date:   Wed Jun 29 22:30:39 2022 +0200

    [Ada] Refine type for checking number of pragma arguments
    
    Code cleanup.
    
    gcc/ada/
    
            * par-prag.adb (Check_Arg_Count): Change parameter type from Int
            to Nat, because this parameter is compared to Arg_Count variable
            which is of type Nat. Also, it wouldn't make sense to check for
            negative number of pragma arguments.

Diff:
---
 gcc/ada/par-prag.adb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gcc/ada/par-prag.adb b/gcc/ada/par-prag.adb
index 88f27f0fed7..e1cf5ba8222 100644
--- a/gcc/ada/par-prag.adb
+++ b/gcc/ada/par-prag.adb
@@ -65,7 +65,7 @@ function Prag (Pragma_Node : Node_Id; Semi : Source_Ptr) return Node_Id is
    --  the routine for the argument one past the last present argument, but
    --  that is the only case in which a non-present argument can be referenced.
 
-   procedure Check_Arg_Count (Required : Int);
+   procedure Check_Arg_Count (Required : Nat);
    --  Check argument count for pragma = Required. If not give error and raise
    --  Error_Resync.
 
@@ -155,7 +155,7 @@ function Prag (Pragma_Node : Node_Id; Semi : Source_Ptr) return Node_Id is
    -- Check_Arg_Count --
    ---------------------
 
-   procedure Check_Arg_Count (Required : Int) is
+   procedure Check_Arg_Count (Required : Nat) is
    begin
       if Arg_Count /= Required then
          Error_Msg_N ("wrong number of arguments for pragma%", Pragma_Node);


                 reply	other threads:[~2022-07-13 10:02 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220713100240.D33323851A9D@sourceware.org \
    --to=pmderodat@gcc.gnu.org \
    --cc=gcc-cvs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).