Source location format in error messages

[nickk-dv]

Here's what current implementation looks like (defined in src/build_settings.cpp):

// temporary
gb_internal char *token_pos_to_string(TokenPos const &pos) {
	gbString s = gb_string_make_reserve(temporary_allocator(), 128);
	String file = get_file_path_string(pos.file_id);
	switch (build_context.ODIN_ERROR_POS_STYLE) {
	default: /*fallthrough*/
	case ErrorPosStyle_Default:
		s = gb_string_append_fmt(s, "%.*s(%d:%d)", LIT(file), pos.line, pos.column);
		break;
	case ErrorPosStyle_Unix:
		s = gb_string_append_fmt(s, "%.*s:%d:%d:", LIT(file), pos.line, pos.column);
		break;
	}
	return s;
}

By default this format is used: dir/file.ext(line:col)

From my experience, this default format style does not work in vs-code on windows (cannot ctrl-click to jump to location).
Rustc uses this format dir/file.ext:line:col example src/main.odin:10:25, which is almost like ErrorPosStyle_Unix but without trailing :.

Whats the reason for having ErrorPosStyle_Default as default? And is it possible to have rust-like format or at least make ErrorPosStyle_Unix the default for all targets in odin releases?

My personal opinion is that rust-like location format (don't know if there's a standard name for it) should be used in all cases to promote consistency and work with vs-code.