X-Git-Url: http://git.efficios.com/?p=babeltrace.git;a=blobdiff_plain;f=converter%2Fbabeltrace-log.c;h=dfcfde71c9697b3a74341969d59e057219eb5ea4;hp=b2935ac9fa488cc3406549df69eebea9cf035c47;hb=39592eae8e476dbb016e2b09ef75f4a7b8cc151d;hpb=ca8b2b0219410977c3e27592f93b8d2e12ec5065

diff --git a/converter/babeltrace-log.c b/converter/babeltrace-log.c
index b2935ac9..dfcfde71 100644
--- a/converter/babeltrace-log.c
+++ b/converter/babeltrace-log.c
@@ -35,7 +35,7 @@
 #include <babeltrace/babeltrace.h>
 #include <babeltrace/ctf/types.h>
 
-#define NSEC_PER_SEC 1000000UL
+#define USEC_PER_SEC 1000000UL
 
 #ifndef UUID_STR_LEN
 #define UUID_STR_LEN	37	/* With \0 */
@@ -45,6 +45,7 @@ int babeltrace_debug, babeltrace_verbose;
 
 static char *s_outputname;
 static int s_timestamp;
+static int s_help;
 static uuid_t s_uuid;
 
 /* Metadata format string */
@@ -59,7 +60,7 @@ static const char metadata_fmt[] =
 "	byte_order = %s;\n"		/* be or le */
 "	packet.header := struct {\n"
 "		uint32_t magic;\n"
-"		uint8_t  trace_uuid[16];\n"
+"		uint8_t  uuid[16];\n"
 "	};\n"
 "};\n"
 "\n"
@@ -111,7 +112,7 @@ void write_packet_header(struct ctf_stream_pos *pos, uuid_t uuid)
 	*(uint32_t *) ctf_get_pos_addr(pos) = 0xC1FC1FC1;
 	ctf_move_pos(pos, sizeof(uint32_t) * CHAR_BIT);
 
-	/* trace_uuid */
+	/* uuid */
 	ctf_dummy_pos(pos, &dummy);
 	ctf_align_pos(&dummy, sizeof(uint8_t) * CHAR_BIT);
 	ctf_move_pos(&dummy, 16 * CHAR_BIT);
@@ -154,7 +155,7 @@ void write_event_header(struct ctf_stream_pos *pos, char *line,
 			char **tline, size_t len, size_t *tlen,
 			uint64_t *ts)
 {
-	unsigned long sec, nsec;
+	unsigned long sec, usec;
 	int ret;
 
 	if (!s_timestamp)
@@ -163,13 +164,16 @@ void write_event_header(struct ctf_stream_pos *pos, char *line,
 	/* Only need to be executed on first pass (dummy) */
 	if (pos->dummy) {
 		/* Extract time from input line */
-		ret = sscanf(line, "[%lu.%lu] ", &sec, &nsec);
+		ret = sscanf(line, "[%lu.%lu] ", &sec, &usec);
 		if (ret == 2) {
 			*tline = strchr(line, ']');
-			if ((*tline)[1] == ' ')
+			assert(*tline);
+			(*tline)++;
+			if ((*tline)[0] == ' ') {
 				(*tline)++;
+			}
 			*tlen = len + line - *tline;
-			*ts = (uint64_t) sec * NSEC_PER_SEC + (uint64_t) nsec;
+			*ts = (uint64_t) sec * USEC_PER_SEC + (uint64_t) usec;
 		}
 	}
 	/* timestamp */
@@ -245,10 +249,12 @@ void usage(FILE *fp)
 	fprintf(fp, "\n");
 	fprintf(fp, "Convert for a text log (read from standard input) to CTF.\n");
 	fprintf(fp, "\n");
-	fprintf(fp, "usage : babeltrace-log OUTPUT\n");
+	fprintf(fp, "usage : babeltrace-log [OPTIONS] OUTPUT\n");
 	fprintf(fp, "\n");
 	fprintf(fp, "  OUTPUT                         Output trace path\n");
 	fprintf(fp, "\n");
+	fprintf(fp, "  -t                             With timestamps (format: [sec.usec] string\\n)\n");
+	fprintf(fp, "\n");
 }
 
 static
@@ -259,6 +265,11 @@ int parse_args(int argc, char **argv)
 	for (i = 1; i < argc; i++) {
 		if (!strcmp(argv[i], "-t"))
 			s_timestamp = 1;
+		else if (!strcmp(argv[i], "-h")) {
+			s_help = 1;
+			return 0;
+		} else if (argv[i][0] == '-')
+			return -EINVAL;
 		else
 			s_outputname = argv[i];
 	}
@@ -276,10 +287,16 @@ int main(int argc, char **argv)
 
 	ret = parse_args(argc, argv);
 	if (ret) {
+		fprintf(stdout, "Error: invalid argument.\n");
 		usage(stdout);
 		goto error;
 	}
 
+	if (s_help) {
+		usage(stdout);
+		exit(EXIT_SUCCESS);
+	}
+
 	ret = mkdir(s_outputname, S_IRWXU|S_IRWXG);
 	if (ret) {
 		perror("mkdir");