--- parser3/src/classes/reflection.C	2025/10/04 15:51:54	1.99
+++ parser3/src/classes/reflection.C	2026/04/24 18:30:15	1.102
@@ -11,7 +11,7 @@
 #include "pa_varray.h"
 #include "pa_vobject.h"
 
-volatile const char * IDENT_REFLECTION_C="$Id: reflection.C,v 1.99 2025/10/04 15:51:54 moko Exp $";
+volatile const char * IDENT_REFLECTION_C="$Id: reflection.C,v 1.102 2026/04/24 18:30:15 moko Exp $";
 
 static const String class_type_methoded("methoded");
 
@@ -126,7 +126,7 @@ static void _create(Request& r, MethodPa
 
 static void _classes(Request& r, MethodParams&) {
 	VHash& result=*new VHash;
-	for(HashString<VStateless_class*>::Iterator i(r.classes()); i; i.next()){
+	for(OrderedHashString<VStateless_class*>::Iterator i(r.classes()); i; i.next()){
 		result.hash().put(i.key(), i.value()->get_methods().count()>0 ? new VString(class_type_methoded) : VVoid::get() );
 	}
 	r.write(result);
@@ -475,29 +475,22 @@ static void _mixin(Request& r, MethodPar
 
 	if(params.count()>1)
 		if(HashStringValue* options=params.as_hash(1, "mixin options")) {
-			int valid_options=0;
 			for(HashStringValue::Iterator i(*options); i; i.next() ){
 				String::Body key=i.key();
 				Value* value=i.value();
 				if(key == "to") {
 					vtarget=value;
-					valid_options++;
 				} else if(key == "name") {
 					name=&value->as_string();
-					valid_options++;
 				} else if(key == "methods") {
 					copy_methods=r.process(*value).as_bool();
-					valid_options++;
 				} else if(key == "fields") {
 					copy_fields=r.process(*value).as_bool();
-					valid_options++;
 				} else if(key == "overwrite") {
 					overwrite=r.process(*value).as_bool();
-					valid_options++;
-				}
+				} else
+					throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
 			}
-			if(valid_options!=options->count())
-				throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
 		}
 
 	if(!vtarget)
@@ -543,23 +536,21 @@ static void _override(Request& r, Method
 	const Method* method=j ? j->method : 0;
 	if(!method)
 		throw Exception(PARSER_RUNTIME, 0, "param must be method junction");
+	if(method->native_code)
+		throw Exception(PARSER_RUNTIME, 0, "param must not be native method");
 
 	const String *name=method->name;
 	Value* vtarget=0;
 	if(params.count()>1)
 		if(HashStringValue* options=params.as_hash(1, "override options")) {
-			int valid_options=0;
 			for(HashStringValue::Iterator i(*options); i; i.next() ){
 				if(i.key() == "to") {
 					vtarget=i.value();
-					valid_options++;
 				} else if(i.key() == "name") {
 					name=&i.value()->as_string();
-					valid_options++;
-				}
+				} else
+					throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
 			}
-			if(valid_options!=options->count())
-				throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
 		}
 
 	if(!vtarget)
@@ -616,28 +607,20 @@ static void _stack(Request& r, MethodPar
 
 	if(params.count()>0)
 		if(HashStringValue* options=params.as_hash(0, "stack options")) {
-			int valid_options=0;
 			for(HashStringValue::Iterator i(*options); i; i.next() ){
 				String::Body key=i.key();
 				Value* value=i.value();
-
 				if(key == "args") {
 					show_args=r.process(*value).as_bool();
-					valid_options++;
 				} else if(key == "locals") {
 					show_locals=r.process(*value).as_bool();
-					valid_options++;
 				} else if(key == "limit") {
 					limit=r.process(*value).as_int();
-					valid_options++;
 				} else if(key == "offset") {
 					offset=r.process(*value).as_int();
-					valid_options++;
-				}
+				} else
+					throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
 			}
-
-			if(valid_options!=options->count())
-				throw Exception(PARSER_RUNTIME, 0, CALLED_WITH_INVALID_OPTION);
 		}
 
 	limit+=offset;