fix(WeaselUI): limit to subscript range when processing candidates

Prcuvu · Prcuvu · commit 6b686c717bfa · 2018-02-28T14:53:47.000+08:00
If candidates.size() is greater than MAX_CANDIDATES_COUNT, only MAX_CANDIDATES_COUNT candidates are processed to prevent display error. Fixes #121
diff --git a/WeaselUI/FullScreenLayout.cpp b/WeaselUI/FullScreenLayout.cpp
@@ -59,7 +59,7 @@ void FullScreenLayout::DoLayout(CDCHandle dc)
 	_auxiliaryRect.OffsetRect(offsetX, offsetY);
 	_highlightRect = m_layout->GetHighlightRect();
 	_highlightRect.OffsetRect(offsetX, offsetY);
-	for (int i = 0, n = (int)_context.cinfo.candies.size(); i < n; ++i)
+	for (int i = 0, n = (int)_context.cinfo.candies.size(); i < n && i < MAX_CANDIDATES_COUNT; ++i)
 	{
 		_candidateLabelRects[i] = m_layout->GetCandidateLabelRect(i);
 		_candidateLabelRects[i].OffsetRect(offsetX, offsetY);
diff --git a/WeaselUI/HorizontalLayout.cpp b/WeaselUI/HorizontalLayout.cpp
@@ -40,7 +40,7 @@ void HorizontalLayout::DoLayout(CDCHandle dc)
 
 	/* Candidates */
 	int w = _style.margin_x, h = 0;
-	for (size_t i = 0; i < candidates.size(); i++)
+	for (size_t i = 0; i < candidates.size() && i < MAX_CANDIDATES_COUNT; ++i)
 	{
 		if (i > 0)
 			w += _style.candidate_spacing;
diff --git a/WeaselUI/VerticalLayout.cpp b/WeaselUI/VerticalLayout.cpp
@@ -42,7 +42,7 @@ void VerticalLayout::DoLayout(CDCHandle dc)
 	int comment_shift_width = 0;  /* distance to the left of the candidate text */
 	int max_candidate_width = 0;  /* label + text */
 	int max_comment_width = 0;    /* comment, or none */
-	for (size_t i = 0; i < candidates.size(); i++)
+	for (size_t i = 0; i < candidates.size() && i < MAX_CANDIDATES_COUNT; ++i)
 	{
 		if (i > 0 )
 			height += _style.candidate_spacing;
@@ -86,7 +86,7 @@ void VerticalLayout::DoLayout(CDCHandle dc)
 	width = max(width, max_content_width + 2 * _style.margin_x);
 
 	/* Align comments */
-	for (size_t i = 0; i < candidates.size(); i++)
+	for (size_t i = 0; i < candidates.size() && i < MAX_CANDIDATES_COUNT; ++i)
 		_candidateCommentRects[i].OffsetRect(_style.margin_x + comment_shift_width, 0);
 
 	if (candidates.size())
diff --git a/WeaselUI/WeaselPanel.cpp b/WeaselUI/WeaselPanel.cpp
@@ -215,7 +215,7 @@ bool WeaselPanel::_DrawCandidates(CDCHandle dc)
 	const std::vector<Text> &comments(m_ctx.cinfo.comments);
 	const std::vector<Text> &labels(m_ctx.cinfo.labels);
 
-	for (size_t i = 0; i < candidates.size(); i++)
+	for (size_t i = 0; i < candidates.size() && i < MAX_CANDIDATES_COUNT; ++i)
 	{
 		CRect rect;
 		if (i == m_ctx.cinfo.highlighted)

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ void FullScreenLayout::DoLayout(CDCHandle dc)`
`59`	`59`	`_auxiliaryRect.OffsetRect(offsetX, offsetY);`
`60`	`60`	`_highlightRect = m_layout->GetHighlightRect();`
`61`	`61`	`_highlightRect.OffsetRect(offsetX, offsetY);`
`62`		`- for (int i = 0, n = (int)_context.cinfo.candies.size(); i < n; ++i)`
	`62`	`+ for (int i = 0, n = (int)_context.cinfo.candies.size(); i < n && i < MAX_CANDIDATES_COUNT; ++i)`
`63`	`63`	`{`
`64`	`64`	`_candidateLabelRects[i] = m_layout->GetCandidateLabelRect(i);`
`65`	`65`	`_candidateLabelRects[i].OffsetRect(offsetX, offsetY);`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ void HorizontalLayout::DoLayout(CDCHandle dc)`
`40`	`40`
`41`	`41`	`/* Candidates */`
`42`	`42`	`int w = _style.margin_x, h = 0;`
`43`		`- for (size_t i = 0; i < candidates.size(); i++)`
	`43`	`+ for (size_t i = 0; i < candidates.size() && i < MAX_CANDIDATES_COUNT; ++i)`
`44`	`44`	`{`
`45`	`45`	`if (i > 0)`
`46`	`46`	`w += _style.candidate_spacing;`
Original file line number	Diff line number	Diff line change
`@@ -215,7 +215,7 @@ bool WeaselPanel::_DrawCandidates(CDCHandle dc)`
`215`	`215`	`const std::vector<Text> &comments(m_ctx.cinfo.comments);`
`216`	`216`	`const std::vector<Text> &labels(m_ctx.cinfo.labels);`
`217`	`217`
`218`		`- for (size_t i = 0; i < candidates.size(); i++)`
	`218`	`+ for (size_t i = 0; i < candidates.size() && i < MAX_CANDIDATES_COUNT; ++i)`
`219`	`219`	`{`
`220`	`220`	`CRect rect;`
`221`	`221`	`if (i == m_ctx.cinfo.highlighted)`