perf: improve entity value range logic for basic variables

[zepfred]

No description provided.

[triceo]

I have to say I don't understand how this does anything for basic variables.

On Slack, you said:

The concern separation property allowed me to create the reachable values for basic variables without any significant updates.

Is this that PR? If so, where exactly does it deal with basic variables?
And if so, why does the enterprise counterpart also only deal with list variables?

I am confused.

[zepfred]

I have to say I don't understand how this does anything for basic variables.

On Slack, you said:

The concern separation property allowed me to create the reachable values for basic variables without any significant updates.

Is this that PR? If so, where exactly does it deal with basic variables? And if so, why does the enterprise counterpart also only deal with list variables?

I am confused.

This PR is still incomplete and only simplifies the ReachableValues class. Regarding the concern separation, I only needed to change the parameter type from ValueRangeManager::getReachableValues to GenuineVariableDescriptor to make it work for basic variables.

[triceo]

This is still in progress.

The PR is now a draft, and that's fair. If it had been a draft from the beginning, I wouldn't have asked some of these questions.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
89.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud