fix(libespm): disable search for 0 formId in CombineBrowser

[Pospelove]

Important

Add check in CombineBrowser::LookupById to return empty result for combFormId 0, preventing irrelevant TES4 records.

• Behavior:
  • In CombineBrowser::LookupById, add check to return empty LookupResult if combFormId is 0, preventing irrelevant TES4 records from being found.

^{This description was created by for 7b22b89. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 7b22b89 in 14 seconds

More details

• Looked at 17 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. libespm/src/CombineBrowser.cpp:25

• Draft comment:
  Consider adding a check for combFormId == 0 in other functions that use combFormId, such as LookupByIdAll, to prevent unnecessary processing.
• Reason this comment was not posted:
  Comment was not on a valid diff hunk.

Workflow ID: wflow_kUiL97ZMNOxzyd41

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.