[PATCH v2 2/4] perf annotate: Period and samples view can't be used at the same time

2017-08-15 Thread Taeung Song
If users give two options --show-total-period
and --show-nr-samples, show their proper usage because
the two options can not be used at the same time.

Cc: Namhyung Kim 
Cc: Jiri Olsa 
Signed-off-by: Taeung Song 
---
 tools/perf/builtin-annotate.c | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/tools/perf/builtin-annotate.c b/tools/perf/builtin-annotate.c
index acde4cc..9d25c27 100644
--- a/tools/perf/builtin-annotate.c
+++ b/tools/perf/builtin-annotate.c
@@ -403,7 +403,7 @@ int cmd_annotate(int argc, const char **argv)
struct perf_data_file file = {
.mode  = PERF_DATA_MODE_READ,
};
-   const struct option options[] = {
+   struct option options[] = {
OPT_STRING('i', "input", _name, "file",
"input file name"),
OPT_STRING('d', "dsos", _conf.dso_list_str, "dso[,dso...]",
@@ -452,8 +452,12 @@ int cmd_annotate(int argc, const char **argv)
 stdio__config_color, "always"),
OPT_END()
};
-   int ret = hists__init();
+   int ret;
+
+   set_option_flag(options, 0, "show-total-period", PARSE_OPT_EXCLUSIVE);
+   set_option_flag(options, 0, "show-nr-samples", PARSE_OPT_EXCLUSIVE);
 
+   ret = hists__init();
if (ret < 0)
return ret;
 
-- 
2.7.4



[PATCH v2 2/4] perf annotate: Period and samples view can't be used at the same time

2017-08-15 Thread Taeung Song
If users give two options --show-total-period
and --show-nr-samples, show their proper usage because
the two options can not be used at the same time.

Cc: Namhyung Kim 
Cc: Jiri Olsa 
Signed-off-by: Taeung Song 
---
 tools/perf/builtin-annotate.c | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/tools/perf/builtin-annotate.c b/tools/perf/builtin-annotate.c
index acde4cc..9d25c27 100644
--- a/tools/perf/builtin-annotate.c
+++ b/tools/perf/builtin-annotate.c
@@ -403,7 +403,7 @@ int cmd_annotate(int argc, const char **argv)
struct perf_data_file file = {
.mode  = PERF_DATA_MODE_READ,
};
-   const struct option options[] = {
+   struct option options[] = {
OPT_STRING('i', "input", _name, "file",
"input file name"),
OPT_STRING('d', "dsos", _conf.dso_list_str, "dso[,dso...]",
@@ -452,8 +452,12 @@ int cmd_annotate(int argc, const char **argv)
 stdio__config_color, "always"),
OPT_END()
};
-   int ret = hists__init();
+   int ret;
+
+   set_option_flag(options, 0, "show-total-period", PARSE_OPT_EXCLUSIVE);
+   set_option_flag(options, 0, "show-nr-samples", PARSE_OPT_EXCLUSIVE);
 
+   ret = hists__init();
if (ret < 0)
return ret;
 
-- 
2.7.4