Thread (11 messages) flat view 11 messages, 2 authors, 2016-06-15

Re: [PATCH v3 3/8] wt-status: move wt_status_colors[] into wt_status structure

From: Jeff King <hidden>
Date: 2016-06-15 22:47:12

On Mon, Aug 10, 2009 at 01:54:20AM -0700, Junio C Hamano wrote:
quoted hunk ↗ jump to hunk
@@ -37,6 +38,7 @@ struct wt_status {
 	int wt_status_relative_paths;
 	int wt_status_submodule_summary;
 	enum untracked_status_type show_untracked_files;
+	char wt_status_colors[6][COLOR_MAXLEN];
Yuck on the magic "6". It should be safe to use

  ARRAY_SIZE(wt_status_colors_default)

since that is computed purely from sizeof's. Though I am not discounting
the possibility that there are compilers which will screw it up.

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help