{"comment":{"created_at":"2011-10-11T08:01:30Z","body":"[artem] Well spotted, a patch would be great! I'd go with a second approach (2), thanks.","updated_at":"2011-10-11T08:01:30Z","url":"https://api.github.com/repos/mapnik-org/mapnik/issues/comments/2358111","id":2358111,"user":{"avatar_url":"https://secure.gravatar.com/avatar/941f68386168a8a6b60f2529a7830af7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/artemp","id":59056,"login":"artemp"}},"action":"created","issue":{"number":550,"created_at":"2011-10-11T08:01:28Z","pull_request":{"diff_url":null,"patch_url":null,"html_url":null},"body":"''mapnik::rule'' class is implemented as a template, however there is the only one specialization of ''rule'' across the whole project (aka ''rule_type'') and template argument isn't in use at all.\r\ntemplate-ness of ''rule'' class should be reconsidered and eliminated if possible (changing all the references to ''rule_type''  into ''rule'')","comments":4,"title":"mapnik::rule redundancy","updated_at":"2011-10-11T08:01:31Z","url":"https://api.github.com/repos/mapnik-org/mapnik/issues/550","id":1872181,"assignee":null,"milestone":null,"closed_at":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/941f68386168a8a6b60f2529a7830af7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/artemp","id":59056,"login":"artemp"},"html_url":"https://github.com/mapnik-org/mapnik/issues/550","labels":[{"name":"Core_Library","url":"https://api.github.com/repos/mapnik-org/mapnik/labels/Core_Library","color":"FFFFFF"}],"state":"open"},"legacy":{"issue_id":1872181,"comment_id":2358111}}