Feature #10904
closedForbid empty aspects
0%
Description
Accidentally we revealed a bug when using an empty aspect. Weaver failed because of this empty aspect was first in the list of aspects for concatenation while it examined the last line from all aspects to append a new line symbol for files that do not end up with it. From the other side, it has no sense to provide empty aspects at all since they just occupy redundant place and do nothing useful. Some users can think that all C models should have the appropriate aspects, but this is not the case. If one does not need to weave in anything, e.g. when one provide function definitions for external undefined functions, it is enough to develop C models without aspects.
Updated by Evgeny Novikov over 3 years ago
- Status changed from New to Resolved
I implemented this issue in branch forbid-empty-aspects. BTW, Core will fail just when somebody will provide empty aspects for models. If empty aspects will be generated by some Core components, Weaver will skip them silently. Let's wait for CI.
Updated by Evgeny Novikov over 3 years ago
- Status changed from Resolved to Closed
Tests passed, so, I merged the branch to master in 453f00506. BTW, all aspects included into the Klever repository are not empty since they at least have the copyright header. The error could happen during development under quite specific conditions as I pointed out in the issue description.