Parser does not honour cluster naming convention

[Gwaptiva]

First off, a great thank you for a wonderful tool. I may not be a fan of the builder-style API, (probably too dumb to use it), so I build a string to feed the parser, and then manipulate the resultant object when I have to.

Here a suggestion for a small improvement. Not a bug, not critical either.

Parsing a DOT file/string like
digraph L { subgraph cluster_root { .... } }

The parser correctly creates me a MutableGraph with a nested MutableGraph, but the nested graph's isCluster method teturns false.

I realise that clusters aren't a DOT/Graphviz standard, but they are listed in the language spec as a convention. Would be nice if the parser honoured this.

Thanks,

Jaap

[nidi3]

A little thing, but a bug never the less. It's fixed in version 0.8.7.

[Gwaptiva]

I'm sure you like low hanging fruits as much as the next one. Thanks for the quick turnaround

[Gwaptiva]

Hmm, looking at your solution: by stripping out the 'cluster_' from the name, will the Graphviz engine still recognise it as a cluster? or does that ignore it anyway?

[nidi3]

It will be added before feeding it to graphviz:

graphviz-java/graphviz-java/src/main/java/guru/nidi/graphviz/model/Serializer.java

Lines 82 to 84 in ed139d2

	str.append("subgraph ")
	.append(Label.of((graph.cluster ? "cluster_" : "") + graph.name).serialized())
	.append(' ');