Rename Lookup* functions for clarity

README.md

@@ -59,7 +59,7 @@ foreach (var child in entities.GetChildren()) {
   pos = new(pos.X * 10, pos.Y * 10);
 }
 
-var onUpdate = world.LookupOrThrow("/flecs/pipeline/OnUpdate");
+var onUpdate = world.LookupByPathOrThrow("/flecs/pipeline/OnUpdate");
 
 // Create a system that will move all entities with
 // the "Position" component downwards by 2 every frame.

src/gaemstone.ECS/Entity.cs

@@ -19,7 +19,9 @@ public readonly struct Entity
 	public bool Equals(Entity other) => Value.Data == other.Value.Data;
 	public override bool Equals(object? obj) => (obj is Entity other) && Equals(other);
 	public override int GetHashCode() => Value.Data.GetHashCode();
-	public override string? ToString() => $"Entity(0x{Value.Data.Data:X})";
+	public override string? ToString()
+		=> IsSome ? $"Entity(0x{Value.Data.Data:X})"
+		          : "Entity.None";
 
 	public static bool operator ==(Entity left, Entity right) =>  left.Equals(right);
 	public static bool operator !=(Entity left, Entity right) => !left.Equals(right);

src/gaemstone.ECS/EntityBase.cs

@@ -9,23 +9,23 @@ public abstract class EntityBase<TReturn>
 	public abstract TReturn Remove(Identifier id);
 	public abstract bool Has(Identifier id);
 
-	public TReturn Add(string symbol) => Add(World.LookupSymbolOrThrow(symbol));
-	public TReturn Add<T>() => Add(World.LookupOrThrow(typeof(T)));
+	public TReturn Add(string symbol) => Add(World.LookupBySymbolOrThrow(symbol));
+	public TReturn Add<T>() => Add(World.LookupByTypeOrThrow(typeof(T)));
 	public TReturn Add(Entity relation, Entity target) => Add(Identifier.Pair(relation, target));
-	public TReturn Add<TRelation>(Entity target) => Add(World.LookupOrThrow<TRelation>(), target);
-	public TReturn Add<TRelation, TTarget>() => Add(World.LookupOrThrow<TRelation>(), World.LookupOrThrow<TTarget>());
+	public TReturn Add<TRelation>(Entity target) => Add(World.LookupByTypeOrThrow<TRelation>(), target);
+	public TReturn Add<TRelation, TTarget>() => Add(World.LookupByTypeOrThrow<TRelation>(), World.LookupByTypeOrThrow<TTarget>());
 
-	public TReturn Remove(string symbol) => Remove(World.LookupSymbolOrThrow(symbol));
-	public TReturn Remove<T>() => Remove(World.LookupOrThrow(typeof(T)));
+	public TReturn Remove(string symbol) => Remove(World.LookupBySymbolOrThrow(symbol));
+	public TReturn Remove<T>() => Remove(World.LookupByTypeOrThrow(typeof(T)));
 	public TReturn Remove(Entity relation, Entity target) => Remove(Identifier.Pair(relation, target));
-	public TReturn Remove<TRelation>(Entity target) => Remove(World.LookupOrThrow<TRelation>(), target);
-	public TReturn Remove<TRelation, TTarget>() => Remove(World.LookupOrThrow<TRelation>(), World.LookupOrThrow<TTarget>());
+	public TReturn Remove<TRelation>(Entity target) => Remove(World.LookupByTypeOrThrow<TRelation>(), target);
+	public TReturn Remove<TRelation, TTarget>() => Remove(World.LookupByTypeOrThrow<TRelation>(), World.LookupByTypeOrThrow<TTarget>());
 
-	public bool Has(string symbol) => Has(World.LookupSymbolOrThrow(symbol));
-	public bool Has<T>() => Has(World.LookupOrThrow(typeof(T)));
+	public bool Has(string symbol) => Has(World.LookupBySymbolOrThrow(symbol));
+	public bool Has<T>() => Has(World.LookupByTypeOrThrow(typeof(T)));
 	public bool Has(Entity relation, Entity target) => Has(Identifier.Pair(relation, target));
-	public bool Has<TRelation>(Entity target) => Has(World.LookupOrThrow<TRelation>(), target);
-	public bool Has<TRelation, TTarget>() => Has(World.LookupOrThrow<TRelation>(), World.LookupOrThrow<TTarget>());
+	public bool Has<TRelation>(Entity target) => Has(World.LookupByTypeOrThrow<TRelation>(), target);
+	public bool Has<TRelation, TTarget>() => Has(World.LookupByTypeOrThrow<TRelation>(), World.LookupByTypeOrThrow<TTarget>());
 
 
 	public abstract T Get<T>(Identifier id);
@@ -36,24 +36,24 @@ public abstract class EntityBase<TReturn>
 	public abstract ref T GetRefOrThrow<T>(Identifier id) where T : unmanaged;
 	public abstract void Modified<T>(Identifier id);
 
-	public T Get<T>() => Get<T>(World.LookupOrThrow<T>());
-	public T? GetOrNull<T>() where T : unmanaged => GetOrNull<T>(World.LookupOrThrow<T>());
-	public T? GetOrNull<T>(T _ = null!) where T : class => GetOrNull<T>(World.LookupOrThrow<T>());
-	public ref T GetMut<T>() where T : unmanaged => ref GetMut<T>(World.LookupOrThrow<T>());
-	public ref T GetRefOrNull<T>() where T : unmanaged => ref GetRefOrNull<T>(World.LookupOrThrow<T>());
-	public ref T GetRefOrThrow<T>() where T : unmanaged => ref GetRefOrThrow<T>(World.LookupOrThrow<T>());
-	public void Modified<T>() => Modified<T>(World.LookupOrThrow<T>());
+	public T Get<T>() => Get<T>(World.LookupByTypeOrThrow<T>());
+	public T? GetOrNull<T>() where T : unmanaged => GetOrNull<T>(World.LookupByTypeOrThrow<T>());
+	public T? GetOrNull<T>(T _ = null!) where T : class => GetOrNull<T>(World.LookupByTypeOrThrow<T>());
+	public ref T GetMut<T>() where T : unmanaged => ref GetMut<T>(World.LookupByTypeOrThrow<T>());
+	public ref T GetRefOrNull<T>() where T : unmanaged => ref GetRefOrNull<T>(World.LookupByTypeOrThrow<T>());
+	public ref T GetRefOrThrow<T>() where T : unmanaged => ref GetRefOrThrow<T>(World.LookupByTypeOrThrow<T>());
+	public void Modified<T>() => Modified<T>(World.LookupByTypeOrThrow<T>());
 
 
 	public abstract TReturn Set<T>(Identifier id, in T value) where T : unmanaged;
 	public abstract TReturn Set<T>(Identifier id, T obj) where T : class;
 
-	public TReturn Set<T>(in T value) where T : unmanaged => Set(World.LookupOrThrow<T>(), value);
-	public TReturn Set<T>(T obj) where T : class => Set(World.LookupOrThrow<T>(), obj);
+	public TReturn Set<T>(in T value) where T : unmanaged => Set(World.LookupByTypeOrThrow<T>(), value);
+	public TReturn Set<T>(T obj) where T : class => Set(World.LookupByTypeOrThrow<T>(), obj);
 
 
 	public TReturn ChildOf(Entity parent) => Add(World.ChildOf, parent);
-	public TReturn ChildOf<TParent>() => Add(World.ChildOf, World.LookupOrThrow<TParent>());
+	public TReturn ChildOf<TParent>() => Add(World.ChildOf, World.LookupByTypeOrThrow<TParent>());
 
 	public TReturn Disable() => Add(World.Disabled);
 	public TReturn Enable() => Remove(World.Disabled);

src/gaemstone.ECS/EntityRef.cs

@@ -50,10 +50,10 @@ public unsafe class EntityRef
 
 	public EntityBuilder NewChild(EntityPath? path = null)
 		=> World.New(EnsureRelativePath(path)).ChildOf(this);
-	public EntityRef? Lookup(EntityPath path)
-		=> World.Lookup(this, EnsureRelativePath(path)!);
-	public EntityRef LookupOrThrow(EntityPath path)
-		=> World.LookupOrThrow(this, EnsureRelativePath(path)!);
+	public EntityRef? LookupChild(EntityPath path)
+		=> World.LookupByPath(this, EnsureRelativePath(path)!);
+	public EntityRef LookupChildOrThrow(EntityPath path)
+		=> World.LookupByPathOrThrow(this, EnsureRelativePath(path)!);
 
 	private static EntityPath? EnsureRelativePath(EntityPath? path)
 		{ if (path?.IsAbsolute == true) throw new ArgumentException("path must not be absolute", nameof(path)); return path; }
@@ -142,9 +142,9 @@ public unsafe class EntityRef
 	public EntityRef? GetTarget(Entity relation, int index = 0)
 		=> CreateOrNull(World, new(ecs_get_target(World, this, relation, index)));
 	public EntityRef? GetTarget(string symbol, int index = 0)
-		=> GetTarget(World.LookupSymbolOrThrow(symbol), index);
+		=> GetTarget(World.LookupBySymbolOrThrow(symbol), index);
 	public EntityRef? GetTarget<T>(int index = 0)
-		=> GetTarget(World.LookupOrThrow(typeof(T)), index);
+		=> GetTarget(World.LookupByTypeOrThrow(typeof(T)), index);
 
 	public bool Equals(EntityRef? other) => (other is not null) && (World == other.World) && (Entity == other.Entity);
 	public override bool Equals(object? obj) => Equals(obj as EntityRef);

src/gaemstone.ECS/IdentifierRef.cs

@@ -27,10 +27,10 @@ public unsafe class IdentifierRef
 		=> new(target.World, Identifier.Pair(relation, target));
 
 	public EntityRef? AsEntity()
-		=> (Flags == default) ? World.Lookup(new Entity(new() { Data = Id })) : null;
+		=> (Flags == default) ? World.LookupAlive(new Entity(new() { Data = Id })) : null;
 	public (EntityRef Relation, EntityRef Target)? AsPair()
-		=> IsPair && (World.Lookup(Id.RelationUnsafe) is EntityRef relation) &&
-		             (World.Lookup(Id.TargetUnsafe  ) is EntityRef target  )
+		=> IsPair && (World.LookupAlive(Id.RelationUnsafe) is EntityRef relation) &&
+		             (World.LookupAlive(Id.TargetUnsafe  ) is EntityRef target  )
 			? (relation, target) : null;
 
 	public bool Equals(IdentifierRef? other) => (other is not null) && (World == other.World) && (Id == other.Id);

src/gaemstone.ECS/Iterator.cs

@@ -98,7 +98,7 @@ public unsafe class Iterator
 	{
 		fixed (ecs_iter_t* ptr = &Value) {
 			var id   = ecs_field_id(ptr, index);
-			var comp = World.LookupOrThrow<T>();
+			var comp = World.LookupByTypeOrThrow<T>();
 			return id == comp.Entity.Value.Data;
 		}
 	}

src/gaemstone.ECS/World+Lookup.cs

@@ -20,38 +20,40 @@ public unsafe partial class World
 			$"Lookup for {type} does not exist"); }
 
 
-	private EntityRef? CreateOrNull(Entity entity)
-		=> EntityRef.CreateOrNull(this, entity);
+	public EntityRef? LookupByType<T>()
+		=> LookupByType(typeof(T));
+	public EntityRef? LookupByType(Type type)
+		=> LookupAlive(_lookupByType.GetValueOrDefault(type));
+	public EntityRef LookupByTypeOrThrow<T>()
+		=> LookupByTypeOrThrow(typeof(T));
+	public EntityRef LookupByTypeOrThrow(Type type)
+		=> LookupByType(type) ?? throw new EntityNotFoundException(
+			$"Entity of type {type} not found");
 
-	public EntityRef? Lookup<T>()
-		=> Lookup(typeof(T));
-	public EntityRef? Lookup(Type type)
-		=> Lookup(_lookupByType.GetValueOrDefault(type));
+	public EntityRef? LookupAlive(Entity value)
+		=> EntityRef.CreateOrNull(this, new(ecs_get_alive(this, value)));
+	public EntityRef LookupAliveOrThrow(Entity entity)
+		=> LookupAlive(entity) ?? throw new EntityNotFoundException(
+			$"Entity {entity} is not alive");
 
-	public EntityRef? Lookup(Entity value)
-		=> CreateOrNull(new(ecs_get_alive(this, value)));
+	public EntityRef? LookupByPath(EntityPath path)
+		=> LookupByPath(default, path);
+	public EntityRef? LookupByPath(Entity parent, EntityPath path)
+		=> EntityRef.CreateOrNull(this, EntityPath.Lookup(this, parent, path, false));
+	public EntityRef LookupByPathOrThrow(EntityPath path)
+		=> LookupByPathOrThrow(default, path);
+	public EntityRef LookupByPathOrThrow(Entity parent, EntityPath path)
+		=> new(this, EntityPath.Lookup(this, parent, path, true));
 
-	public EntityRef? Lookup(EntityPath path)
-		=> Lookup(default, path);
-	public EntityRef? Lookup(Entity parent, EntityPath path)
-		=> CreateOrNull(EntityPath.Lookup(this, parent, path));
-	public EntityRef? LookupSymbol(string symbol)
+	public EntityRef? LookupBySymbol(string symbol)
 	{
 		using var alloc = TempAllocator.Use();
-		return CreateOrNull(new(ecs_lookup_symbol(this, alloc.AllocateCString(symbol), false)));
+		var entity = ecs_lookup_symbol(this, alloc.AllocateCString(symbol), false);
+		return EntityRef.CreateOrNull(this, new(entity));
 	}
-
-	public EntityRef LookupOrThrow<T>() => LookupOrThrow(typeof(T));
-	public EntityRef LookupOrThrow(Type type) => Lookup(type)
-		?? throw new EntityNotFoundException($"Entity of type {type} not found");
-	public EntityRef LookupOrThrow(Entity entity) => Lookup(entity)
-		?? throw new EntityNotFoundException($"Entity {entity} not alive");
-	public EntityRef LookupOrThrow(EntityPath path) => Lookup(default, path)
-		?? throw new EntityNotFoundException($"Entity '{path}' not found");
-	public EntityRef LookupOrThrow(Entity parent, EntityPath path) => Lookup(parent, path)
-		?? throw new EntityNotFoundException($"Child entity of {parent} '{path}' not found");
-	public EntityRef LookupSymbolOrThrow(string symbol) => LookupSymbol(symbol)
-		?? throw new EntityNotFoundException($"Entity with symbol '{symbol}' not found");
+	public EntityRef LookupBySymbolOrThrow(string symbol)
+		=> LookupBySymbol(symbol) ?? throw new EntityNotFoundException(
+			$"Entity with symbol '{symbol}' not found");
 
 
 	public class EntityNotFoundException : Exception

src/gaemstone.ECS/World.cs

@@ -19,9 +19,9 @@ public unsafe partial class World
 	{
 		Handle = ecs_init_w_args(args.Length, null);
 
-		ChildOf   = LookupOrThrow("/flecs/core/ChildOf");
-		Disabled  = LookupOrThrow("/flecs/core/Disabled");
-		DependsOn = LookupOrThrow("/flecs/core/DependsOn");
+		ChildOf   = LookupByPathOrThrow("/flecs/core/ChildOf");
+		Disabled  = LookupByPathOrThrow("/flecs/core/Disabled");
+		DependsOn = LookupByPathOrThrow("/flecs/core/DependsOn");
 	}
 
 	public void Dispose()