openmw.org

I'd suggest to run the Coverity Scan again before 0.45 RC and deal with new issues which it will find.

We still have a regression on Mac OS: https://gitlab.com/OpenMW/openmw/issues/4665

Zini wrote: ↑

09 Oct 2018, 13:51

Okay, that sounds reasonable.

Ok, then I'll implement it later as a separate pull request, if my current one with scabbards and quivers will be merged. Otherwise this PR can become a quite big.

If we really want to implement shields sheathing, I suggest this approach: 1. Check if shield model has the _sh file. If not, handle it as we handle shields now. 2. If the shield has sheath model, place it on character's back when using a one-handed weapon and weapon is holstered. When using two-han...

In theory, if we are not going to add shields holstering for now, there should not be much issues with mentioned mods.

In theory, it is possible to add a hack - if a shield has a zero AR, or AR >= 100, consider it as a hold item instead of shield, but I do not like it much.

There is a question about how must-have things (for example, weapon sheathing, glowing windows, animated containers, etc.) ever should be implemented in OpenMW. Should we have such things out of box without need to use a 3d-party code, our we should leave it to Lua scripting to make these features m...

What operations would there be to do? Other then adding a new field? Which is in fact trivial (it happens automatically). Limit input field range, use tabs to do not waste space, or add a custom logic (for example, autocalculate spell cost when user checks Auto Calc checkbox, or add an encumberance...

For now even trivial operations require a good understanding about how these auto-generated tables work, because this feature is extremely unfriendly for new developers, even if they have knowledge about how Qt works. Also if there is a need to do something which this autogeneration framework does n...