Review pass: fix stale comments (drop mountain refs, update AnimalSet summary)
Code review of the session''s work found the simulation deterministic, save/load complete, def/localization consistent, and no real correctness bugs (the rest of the flagged items were defensive-paranoia false positives). Only fixes needed were stale comments: AnimalSet still described the Phase-A1 single-sprite era (now directional sprites + stages/sex), and two occluder comments referenced the removed mountains. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 4.8
parent
65ecd2c7b2
commit
d196437462
@@ -8,7 +8,8 @@ namespace LittleSim.Content;
|
||||
/// Готовая (resolved) таблица животных для сцены: по каждому <see cref="AnimalDef"/> — загруженный
|
||||
/// регион атласа и набор генов вида (<see cref="GenomeTemplate"/>) для генерации особей. Строится
|
||||
/// один раз при загрузке сцены (нужны атлас и устройство), как <see cref="PlantSet"/>. Системы и
|
||||
/// фабрика индексируются в неё без словарей. Фаза A1 — один спрайт на вид; стадии/пол придут позже.
|
||||
/// фабрика индексируются в неё без словарей. Несёт направленные спрайты (восток/север/юг + флип) по
|
||||
/// полу/стадии, спрайты трупа/яйца, анатомию, шаблон генома и предпосчёты для горячих путей.
|
||||
/// </summary>
|
||||
public sealed class AnimalSet
|
||||
{
|
||||
|
||||
@@ -27,7 +27,7 @@ public sealed class TerrainDef : Def
|
||||
/// <summary>Суша: здесь появляются жители, животные и растения.</summary>
|
||||
public bool IsLand { get; init; }
|
||||
|
||||
/// <summary>Загораживает свет (горы) — базовый окклюдер для лайтмапа/теней.</summary>
|
||||
/// <summary>Светонепроницаемый террейн — базовый окклюдер лайтмапа/теней (сейчас такого нет).</summary>
|
||||
public bool BlocksLight { get; init; }
|
||||
|
||||
/// <summary>Ключ текстуры поверхности для тайловой сцены; null — тонированный тайл.</summary>
|
||||
|
||||
@@ -994,7 +994,7 @@ public sealed class WorldScene : Scene
|
||||
return shore.ToArray();
|
||||
}
|
||||
|
||||
// Текущая сетка окклюдеров для лайтмапа: статичные горы + клетки со зрелыми деревьями.
|
||||
// Текущая сетка окклюдеров для лайтмапа: светонепроницаемый террейн (сейчас нет) + зрелые деревья.
|
||||
private bool[] BuildOccluders()
|
||||
{
|
||||
Array.Copy(_cellOccluderBase, _cellOccluder, _cellOccluder.Length);
|
||||
|
||||
Reference in New Issue
Block a user