From d9526e4c768213fdea66c850fe41363196dd68f8 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Tue, 29 Jun 2021 20:51:30 +0300
Subject: [PATCH] several minor tweaks...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 README.md    |  7 +++++++
 object.js    | 28 +++++++++++++++++++---------
 package.json |  2 +-
 3 files changed, 27 insertions(+), 10 deletions(-)

diff --git a/README.md b/README.md
index 5a45179..6e9dac0 100755
--- a/README.md
+++ b/README.md
@@ -619,6 +619,13 @@ create(<base>)
     -> <obj>
 ```
 
+For functions we can set `.name`
+```bnf
+create(<name>, <base-func>)
+    -> <func>
+```
+
+
 This is similar to [`Object.create(..)`] but handles callables correctly, i.e. if 
 `<base>` is a callable then `<obj>` will also be callable.
 
diff --git a/object.js b/object.js
index 0ac89a7..867e9fb 100755
--- a/object.js
+++ b/object.js
@@ -273,9 +273,12 @@ var create =
 module.create =
 function(obj){
 	// name given...
+	var name = ''
 	if(typeof(obj) == 'string' && arguments.length > 1){
-		var name
-		;[name, obj] = arguments }
+		;[name, obj] = arguments 
+		// sanity check...
+		if(!/^[a-zA-Z_][a-zA-Z0-9_]*$/.test(name)){
+			throw new Error(`create(..): invalid name: "${name}"`) } }
 	// calable...
 	if(typeof(obj) == 'function'){
 		var func = function(){
@@ -286,10 +289,12 @@ function(obj){
 				// NOTE: if obj does not inherit from Function .call 
 				// 		might not be available...
 				: Function.prototype.call.call(obj, func, ...arguments) }
-
-		// XXX set the name....
-		// XXX
-
+		func.name = name
+		func.name != name
+			&& (func = eval('('+ 
+				func
+					.toString()
+					.replace(/function\(/, `function ${name}(`) +')'))
 		func.__proto__ = obj
 		// XXX not sure about this yet...
 		Object.defineProperty(func, 'toString', {
@@ -937,6 +942,10 @@ module.C =
 function Constructor(name, a, b, c){
 	var args = [...arguments].slice(1, 4)
 
+	// sanity check...
+	if(!/^[a-zA-Z_][a-zA-Z0-9_]*$/.test(name)){
+		throw new Error(`Constructor(..): invalid name: "${name}"`) }
+
 	// parse args...
 	// 	Constructor(name[[, constructor[, mixin]], proto])
 	var proto = args.pop() || {}
@@ -1007,10 +1016,11 @@ function Constructor(name, a, b, c){
 	// NOTE: this eval(..) should not be a risk as its inputs are
 	// 		static and never infuenced by external inputs...
 	// NOTE: this will fail with non-identifier names...
-	_constructor.name == 'Constructor'
-		&& eval('_constructor = '+ _constructor
+	_constructor.name != name
+		&& (_constructor = eval('('+ 
+			_constructor
 				.toString()
-				.replace(/Constructor/g, name))
+				.replace(/Constructor/g, name) +')'))
 	// set .toString(..)...
 	// NOTE: this test is here to enable mixinFlat(..) to overwrite 
 	// 		.toString(..) below...
diff --git a/package.json b/package.json
index ed36732..3ec34f2 100755
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ig-object",
-  "version": "5.5.1",
+  "version": "5.5.2",
   "description": "",
   "main": "object.js",
   "scripts": {