pl/isql language that support function and procedure(II)

refine the comments.

Author: bigplaice

src/backend/commands/functioncmds.c

@@ -1614,10 +1614,10 @@ CreateFunction(ParseState *pstate, CreateFunctionStmt *stmt)
 }
 
 /*
- * support oracle grammer like
+ * Support oracle grammer:
  * alter function func editionable|noneditionable or compile
- * this function we doesn't consider function arguments and
- * search pg_proc directly, if we found more than one or less
+ * In this function we don't support function arguments or 
+ * searching pg_proc directly, if we found more than one or less
  * than one, we report error.
  */
 ObjectAddress

src/backend/oracle_parser/ora_gram.y

@@ -9104,7 +9104,7 @@ implementation_package:
 			{
 				$$ = NIL;
 			}
-		/* reuse implementation_type temporary */
+		/* reuse implementation_type temporarily */
 		| implementation_package_type POLYMORPHIC implementation_type
 			{
 				$$ = $3;
@@ -9202,7 +9202,7 @@ opt_ora_func_args_with_defaults:
 	 * In order to avoid reduce/reduce conflict, we are compatible with Oracle
 	 * based on the original CREATE PROCEUDRE grammar rules of PG. The purpose
 	 * of adding this nonterminal is to try not to destroy the syntax of PG.
-	 * Although we can switch to the gram.y of native PG, try our best to Stick
+	 * Although we can switch to the gram.y of native PG, try our best to stick
 	 * to this principle.
 	 */
 	opt_procedure_args_with_defaults:

src/include/nodes/parsenodes.h

@@ -4394,21 +4394,21 @@ typedef struct AccessibleByClause
 } AccessibleByClause;
 
 /*
- * support alter a function like
+ * support alter a function like:
  * alter function func editable|noneditable or compile
- * we doesn't use the AlterFunctionStmt, because we doesn't
+ * we don't use the AlterFunctionStmt, because we don't
  * want to use AlterFunction function, which will modify the
  * pg_proc catalog table,compile doesn't change the system catalog.
- * another reason is AlterFunction which fun should
+ * another reason is AlterFunction which should
  * bring function arguments and find function rely on its
- * arguments, this struct will doesn't consider function arguments.
+ * arguments, this struct will not consider function arguments.
  */
 typedef struct CompileFunctionStmt
 {
 	NodeTag		type;
 	ObjectType	objtype;
 	ObjectWithArgs *func;		/* name and args of function */
-	bool		is_compile;		/* if false, it is change the editable|noneditable */
+	bool		is_compile;		
 	bool		editable;
 	List		*parameters;
 } CompileFunctionStmt;

src/include/pg_config.h.in

@@ -33,7 +33,7 @@
 /* Saved arguments from configure */
 #undef CONFIGURE_ARGS
 
-/* Define to the default TCP port number on which the server listens and to
+/* Define the default TCP port number on which the server listens and to
    which clients will try to connect. This can be overridden at run-time, but
    it's convenient if your clients have the right default compiled in.
    (--with-oraport=PORTNUM) */

src/oracle_fe_utils/ora_psqlscan.l

@@ -1869,7 +1869,7 @@ is_oracle_slash(PsqlScanState state, const char *line)
 				len = strlen(line);
 				for (i = 0; i < len; i++)
 				{
-					/* allowed special char */
+					/* allow special char */
 					if (line[i] == '\t' ||
 						line[i] == '\n' ||
 						line[i] == '\r' ||

src/oracle_test/regress/expected/ora_plisql.out

@@ -1,7 +1,7 @@
 --
 -- PLISQL
 --
--- return is a non-reserved keyword, Can be used as object name.
+-- return is a non-reserved keyword, can be used as object name.
 CREATE TABLE RETURN (fooid INT, foosubid INT, fooname TEXT);
 INSERT INTO RETURN VALUES (1, 2, 'three');
 INSERT INTO RETURN VALUES (4, 5, 'six');
@@ -936,7 +936,7 @@ SELECT pg_get_functiondef('test_subproc_func'::regproc) from dual;
  end;
 (1 row)
 
--- ivy_get_plisql_functiondef is only using get plisql func/proc definition.
+-- ivy_get_plisql_functiondef is only used to get plisql func/proc definition.
 SELECT ivy_get_plisql_functiondef('test_func'::regproc) from dual;
           ivy_get_plisql_functiondef           
 -----------------------------------------------
@@ -1142,7 +1142,7 @@ EXEC test_proc6(2, 3, 4);
 (1 row)
 
 DROP PROCEDURE test_proc6;
--- recursive with output arguments
+-- recursion with output arguments
 CREATE OR REPLACE PROCEDURE test_proc7(x int, INOUT a int, INOUT b numeric)
 AS
 BEGIN

src/oracle_test/regress/sql/ora_plisql.sql

@@ -2,7 +2,7 @@
 -- PLISQL
 --
 
--- return is a non-reserved keyword, Can be used as object name.
+-- return is a non-reserved keyword, can be used as object name.
 CREATE TABLE RETURN (fooid INT, foosubid INT, fooname TEXT);
 INSERT INTO RETURN VALUES (1, 2, 'three');
 INSERT INTO RETURN VALUES (4, 5, 'six');
@@ -934,7 +934,7 @@ SELECT pg_get_functiondef('test_func'::regproc) from dual;
 SELECT pg_get_functiondef('test_proc'::regproc) from dual;
 SELECT pg_get_functiondef('test_subproc_func'::regproc) from dual;
 
--- ivy_get_plisql_functiondef is only using get plisql func/proc definition.
+-- ivy_get_plisql_functiondef is only used to get plisql func/proc definition.
 SELECT ivy_get_plisql_functiondef('test_func'::regproc) from dual;
 SELECT ivy_get_plisql_functiondef('test_proc'::regproc) from dual;
 SELECT ivy_get_plisql_functiondef('test_subproc_func'::regproc) from dual;
@@ -1056,7 +1056,7 @@ END;
 CALL test_proc6(2, 3, 4);
 EXEC test_proc6(2, 3, 4);
 DROP PROCEDURE test_proc6;
--- recursive with output arguments
+-- recursion with output arguments
 
 CREATE OR REPLACE PROCEDURE test_proc7(x int, INOUT a int, INOUT b numeric)
 AS