[PATCH v3 3/3] doc: update diagrams for compatibility.

[Bryce Harrington]

On Wed, Jan 28, 2015 at 05:24:06PM -0800, Jon A. Cruz wrote:
> Change attribute separators for compatiblity with graphviz older than 2.30.
> 
> Signed-off-by: Jon A. Cruz <jonc at osg.samsung.com>

Changes look good, and I verified the output comes up fine in firefox,
and actually looks a bit nicer than the current image.

Reviewed-by: Bryce Harrington <bryce at osg.samsung.com>
Tested-by: Bryce Harrington <bryce at osg.samsung.com>

Btw, patch 1/3 never came through to me, although patchwork sees it fine.

> ---
>  doc/doxygen/dot/wayland-architecture.gv | 44 ++++++++++++++---------------
>  doc/doxygen/dot/x-architecture.gv       | 50 ++++++++++++++++-----------------
>  2 files changed, 47 insertions(+), 47 deletions(-)
> 
> diff --git a/doc/doxygen/dot/wayland-architecture.gv b/doc/doxygen/dot/wayland-architecture.gv
> index b86f4b5..2d5db84 100644
> --- a/doc/doxygen/dot/wayland-architecture.gv
> +++ b/doc/doxygen/dot/wayland-architecture.gv
> @@ -1,39 +1,39 @@
>  digraph arch_wayland {
>      edge[
> -        fontname="DejaVu Sans";
> -        dir="both";
> -        arrowtail="dot";
> -        arrowsize=.5;
> -        fontname="DejaVu Sans"
> -        fontsize="18";
> +        fontname="DejaVu Sans",
> +        dir="both",
> +        arrowtail="dot",
> +        arrowsize=.5,
> +        fontname="DejaVu Sans",
> +        fontsize="18",
>      ]
>  
>      node[
> -        shape="Mrecord";
> -        color=none;
> -        fillcolor="#ffbc00";
> -        style="filled";
> -        fontname="DejaVu Sans"
> -        fontsize="18";
> +        shape="Mrecord",
> +        color=none,
> +        fillcolor="#ffbc00",
> +        style="filled",
> +        fontname="DejaVu Sans",
> +        fontsize="18",
>     ]
>  
> -    c1 [label="Wayland Client"; URL="#c1"]
> -    c2 [label="Wayland Client"; URL="#c2"]
> +    c1 [label="Wayland Client", URL="#c1"]
> +    c2 [label="Wayland Client", URL="#c2"]
>  
> -    comp [tooltip="Wayland Compositor" label="|{|Wayland\nCompositor|}|"; URL="#comp"]
> +    comp [tooltip="Wayland Compositor", label="|{|Wayland\nCompositor|}|", URL="#comp"]
>  
> -    impl [tooltip="KMS evdev Kernel" label="|{{KMS|evdev}|Kernel}|"; URL="#impl"]
> +    impl [tooltip="KMS evdev Kernel", label="|{{KMS|evdev}|Kernel}|", URL="#impl"]
>  
>  
> -    c1 -> comp [taillabel="③"; labeldistance=2.5; URL="#step_3"];
> +    c1 -> comp [taillabel="③", labeldistance=2.5, URL="#step_3"];
>      c2 -> comp;
>  
> -    comp -> c1 [label="②"; URL="#step_2"];
> +    comp -> c1 [label="②", URL="#step_2"];
>      comp -> c2;
>  
> -    comp -> impl [xlabel = "④"; URL="#step_4"];
> -    comp -> impl [style = invis; label="    "];
> -    impl -> comp [xlabel = "①"; URL="#step_1"];
> +    comp -> impl [xlabel = "④", URL="#step_4"];
> +    comp -> impl [style = invis, label="    "];
> +    impl -> comp [xlabel = "①", URL="#step_1"];
>  
>      c1 -> c2 [style=invis];
> - }
> +}
> diff --git a/doc/doxygen/dot/x-architecture.gv b/doc/doxygen/dot/x-architecture.gv
> index 85c98a3..4ea49bf 100644
> --- a/doc/doxygen/dot/x-architecture.gv
> +++ b/doc/doxygen/dot/x-architecture.gv
> @@ -1,52 +1,52 @@
>  digraph arch_x {
>      edge[
> -        fontname="DejaVu Sans";
> -        dir="both";
> -        arrowtail="dot";
> -        arrowsize=.5;
> -        fontname="DejaVu Sans"
> -        fontsize="18";
> +        fontname="DejaVu Sans",
> +        dir="both",
> +        arrowtail="dot",
> +        arrowsize=.5,
> +        fontname="DejaVu Sans",
> +        fontsize="18",
>      ]
>  
>      node[
> -        shape="Mrecord";
> -        color=none;
> -        fillcolor="#ffbc00";
> -        style="filled";
> -        fontname="DejaVu Sans"
> -        fontsize="18";
> +        shape="Mrecord",
> +        color=none,
> +        fillcolor="#ffbc00",
> +        style="filled",
> +        fontname="DejaVu Sans",
> +        fontsize="18",
>      ]
>  
>      {
>          rank=same;
> -        c1 [label="X Client"; URL="#c1"]
> -        c3 [label="X Client"; URL="#c3"]
> +        c1 [label="X Client", URL="#c1"]
> +        c3 [label="X Client", URL="#c3"]
>      }
> -    c2 [label="X Client"; URL="#c2"]
> +    c2 [label="X Client", URL="#c2"]
>  
>      {
>          rank=same;
> -        xserver [tooltip="X Server" label="|{|X Server|}|"; URL="#xserver"]
> -        comp [tooltip="Compositor" label="|{|Compositor|}|"; URL="#comp"]
> +        xserver [tooltip="X Server", label="|{|X Server|}|", URL="#xserver"]
> +        comp [tooltip="Compositor", label="|{|Compositor|}|", URL="#comp"]
>      }
>  
> -    impl [tooltip="KMS evdev Kernel" label="|{{KMS|evdev}|Kernel}|"; URL="#impl"]
> +    impl [tooltip="KMS evdev Kernel", label="|{{KMS|evdev}|Kernel}|", URL="#impl"]
>  
> -    c1 -> xserver [taillabel="③"; labeldistance=2; URL="#step_3"];
> +    c1 -> xserver [taillabel="③", labeldistance=2, URL="#step_3"];
>      c2 -> xserver;
>      c3 -> xserver;
>  
> -    xserver -> c1 [taillabel="②"; labeldistance=2; URL="#step_2"];
> +    xserver -> c1 [taillabel="②", labeldistance=2, URL="#step_2"];
>      xserver -> c2;
>      xserver -> c3;
>  
> -    xserver -> impl [taillabel = "⑥"; labeldistance=1.75; URL="#step_6"];
> -    xserver -> impl [style = invis; label="    "];
> -    impl -> xserver [taillabel = "①"; labeldistance=1.75; URL="#step_1"];
> +    xserver -> impl [taillabel="⑥", labeldistance=1.75, URL="#step_6"];
> +    xserver -> impl [style=invis, label="    "];
> +    impl -> xserver [taillabel="①", labeldistance=1.75, URL="#step_1"];
>  
>      xserver -> comp [style=invis];
> -    xserver -> comp [taillabel="④"; labeldistance=1.75; labelangle=-45; URL="#step_4"];
> -    comp -> xserver [taillabel="⑤"; URL="#step_5"];
> +    xserver -> comp [taillabel="④", labeldistance=1.75, labelangle=-45, URL="#step_4"];
> +    comp -> xserver [taillabel="⑤", URL="#step_5"];
>      comp -> xserver [style=invis]
>  
>      c1 -> c2 [style=invis];
> -- 
> 2.1.0
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel