YafaRay Plug-in - reviewing for Yafaray 3.0

[micheus]

David, since the DEF_RENDER_PASS* are defined as disabled, there is no need for that sequence of defines.

This sequence bellow - in export_prefs() - also doesn't need to be that way:

{render_pass1,?DEF_RENDER_PASS1},
{render_pass2,?DEF_RENDER_PASS2},
...

it was my bad use a tuple instead of an atom as id for the new fields as I suggested you:

Code:

RenderPass =
        lists:foldl(fun(N, Acc) ->
                         Acc++[{{pass,N},?DEF_RENDER_PASS}]
                     end, [], lists:seq(1,32)),

In that code I suggested you, replace it by:

Code:

RenderPass =
        lists:foldl(fun(N, Acc) ->
                         Id = io_lib:format("render_pass~w",[N]),
                         Acc++[list_to_atom(Id),?DEF_RENDER_PASS}]
                     end, [], lists:seq(1,32)),

That way you can go back to the logic I tried to suggest before and have less code to write.