diff mbox series

Verbose support in analyze_brprob_spec

Message ID 20211215015857.949170-1-luoxhu@linux.ibm.com
State Committed
Commit e1f8c14b9a0387e2beffeee002c56164166a265b
Headers
Series Verbose support in analyze_brprob_spec |

Commit Message

Xionghu Luo Dec. 15, 2021, 1:58 a.m. UTC 
  Also add verbose argument support like analyze_brprob.py

contrib/ChangeLog:

	* analyze_brprob_spec.py: Add verbose argument.
---
 contrib/analyze_brprob_spec.py | 1 +
 1 file changed, 1 insertion(+)

Comments

Martin Liška Dec. 15, 2021, 10:34 a.m. UTC | #1 
On 12/15/21 02:58, Xionghu Luo wrote:
> Also add verbose argument support like analyze_brprob.py
> 
> contrib/ChangeLog:
> 
> 	* analyze_brprob_spec.py: Add verbose argument.
> ---
>   contrib/analyze_brprob_spec.py | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/contrib/analyze_brprob_spec.py b/contrib/analyze_brprob_spec.py
> index e621853ba4e..063bd11d99c 100755
> --- a/contrib/analyze_brprob_spec.py
> +++ b/contrib/analyze_brprob_spec.py
> @@ -31,6 +31,7 @@ parser.add_argument('-s', '--sorting', dest = 'sorting',
>       choices = ['branches', 'branch-hitrate', 'hitrate', 'coverage', 'name'],
>       default = 'branches')
>   parser.add_argument('-d', '--def-file', help = 'path to predict.def')
> +parser.add_argument('-v', '--verbose', action = 'store_true', help = 'Print verbose informations')

Hello.

Is the argument properly passes to invocation of analyze_brprob.py script?
If so, then please install the patch.

Cheers,
Martin

>   
>   args = parser.parse_args()
>
Xionghu Luo Dec. 16, 2021, 2:14 a.m. UTC | #2 
On 2021/12/15 18:34, Martin Liška wrote:
> On 12/15/21 02:58, Xionghu Luo wrote:
>> Also add verbose argument support like analyze_brprob.py
>>
>> contrib/ChangeLog:
>>
>>     * analyze_brprob_spec.py: Add verbose argument.
>> ---
>>   contrib/analyze_brprob_spec.py | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/contrib/analyze_brprob_spec.py
>> b/contrib/analyze_brprob_spec.py
>> index e621853ba4e..063bd11d99c 100755
>> --- a/contrib/analyze_brprob_spec.py
>> +++ b/contrib/analyze_brprob_spec.py
>> @@ -31,6 +31,7 @@ parser.add_argument('-s', '--sorting', dest =
>> 'sorting',
>>       choices = ['branches', 'branch-hitrate', 'hitrate', 'coverage',
>> 'name'],
>>       default = 'branches')
>>   parser.add_argument('-d', '--def-file', help = 'path to predict.def')
>> +parser.add_argument('-v', '--verbose', action = 'store_true', help =
>> 'Print verbose informations')
> 
> Hello.
> 
> Is the argument properly passes to invocation of analyze_brprob.py script?
> If so, then please install the patch.

Thanks, installed to r12-6010.

> 
> Cheers,
> Martin
> 
>>     args = parser.parse_args()
>>   
> 
>
diff mbox series

Patch

diff --git a/contrib/analyze_brprob_spec.py b/contrib/analyze_brprob_spec.py
index e621853ba4e..063bd11d99c 100755
--- a/contrib/analyze_brprob_spec.py
+++ b/contrib/analyze_brprob_spec.py
@@ -31,6 +31,7 @@  parser.add_argument('-s', '--sorting', dest = 'sorting',
     choices = ['branches', 'branch-hitrate', 'hitrate', 'coverage', 'name'],
     default = 'branches')
 parser.add_argument('-d', '--def-file', help = 'path to predict.def')
+parser.add_argument('-v', '--verbose', action = 'store_true', help = 'Print verbose informations')
 
 args = parser.parse_args()