73 + if (cols == 2) 74 + { 75 + genre_layout_->SetChildrenSize (dash::Style::Instance().GetTileWidth() - 12, 32); 76 + } 77 + else if (cols == 3) 78 + { 79 + genre_layout_->SetChildrenSize (92, 33); 80 + }
Can you add a catch-all else{} branch to this construction, maybe printing a warning? Or if it's unneeded please add a comment that the off-chance is taken care of.
Otherwise looking good.
« Back to merge proposal
73 + if (cols == 2) ->SetChildrenSi ze (dash:: Style:: Instance( ).GetTileWidth( ) - 12, 32); ->SetChildrenSi ze (92, 33);
74 + {
75 + genre_layout_
76 + }
77 + else if (cols == 3)
78 + {
79 + genre_layout_
80 + }
Can you add a catch-all else{} branch to this construction, maybe printing a warning? Or if it's unneeded please add a comment that the off-chance is taken care of.
Otherwise looking good.