json: faust generates duplicate shortname for bargraph used multiple times

[crop2000]

in the following example the bargraph is used twice.
faust identifies that the name is used twice and prefixes the shortname with P1_ but it uses the same prefix for both shortnames.
i expect that short names are unique.

declare name "VolumeControl";

import("stdfaust.lib");

process(l,r) =  (l : meter_l), (r : meter_r) with {
        left_vol =  vbargraph("left",0,1);
        meter_l(l) = attach(l, l : abs  : left_vol );
        meter_r(r) = attach(r, r : abs  : left_vol );
    
};

	"ui": [
		{
			"type": "vgroup",
			"label": "VolumeControl",
			"items": [
				{
					"type": "vbargraph",
					"label": "left",
					"varname": "fVbargraph0",
					"shortname": "P1_VolumeControl_left",
					"address": "/VolumeControl/left",
					"min": 0,
					"max": 1
				},
				{
					"type": "vbargraph",
					"label": "left",
					"varname": "fVbargraph1",
					"shortname": "P1_VolumeControl_left",
					"address": "/VolumeControl/left",
					"min": 0,
					"max": 1
				}
			]
		}
	]

I am on master-dev.
But the problem has not been introduced by recent changes.
It is also present in 2.74...

[sletz]

This is actually a "feature", see here:

faust/compiler/generator/json_instructions.hh

Line 49 in c7ec4c2

const std::string& insertInputsPath(const std::string& path)

and

faust/compiler/generator/json_instructions.hh

Line 63 in c7ec4c2

const std::string& insertOutputsPath(const std::string& path)

I'll add something in the FAQ ASAP.

Look also at: https://faustdoc.grame.fr/manual/errors/#warning-messages

[crop2000]

I still think it is strange that it is prefixed because of the duplicate use it introduced, but the prefix is twice the same.
the issue is solveable if the shortname is not associated with the path but with the varname.

[sletz]

See https://faustdoc.grame.fr/manual/faq/#pathnames-in-the-gui-items

[crop2000]

The documentation doesn't deal with shortnames and the issue is about shortnames.
A pr to solve this issue can be made.
But i think you don't have interest that this gets solved.

[sletz]

shortnames are built for pathnames, so if pathnames are the same, so are shortnames. You can always decide to restructure the code to have different pathnames.

[crop2000]

I think in that case an error should be raised. Because there is no way to access the duplicate path.