Re: [PATCH 2/2] ftrace: Make if condition correctly due to the operator order

2015-07-26 Thread Minfei Huang
On 07/27/15 at 10:20am, yalin wang wrote: > > > On Jul 26, 2015, at 23:55, Minfei Huang wrote: > > > > The if condition will be always true, since the operator & has the high > > priority than operator ||. > > > > Use () to quote them to make the if condition correctly. > > > > Signed-off-by:

Re: [PATCH 2/2] ftrace: Make if condition correctly due to the operator order

2015-07-26 Thread yalin wang
> On Jul 26, 2015, at 23:55, Minfei Huang wrote: > > The if condition will be always true, since the operator & has the high > priority than operator ||. > > Use () to quote them to make the if condition correctly. > > Signed-off-by: Minfei Huang > --- > kernel/trace/ftrace.c | 2 +- > 1 file

[PATCH 2/2] ftrace: Make if condition correctly due to the operator order

2015-07-26 Thread Minfei Huang
The if condition will be always true, since the operator & has the high priority than operator ||. Use () to quote them to make the if condition correctly. Signed-off-by: Minfei Huang --- kernel/trace/ftrace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 2/2] ftrace: Make if condition correctly due to the operator order

2015-07-26 Thread Minfei Huang
The if condition will be always true, since the operator has the high priority than operator ||. Use () to quote them to make the if condition correctly. Signed-off-by: Minfei Huang mnfhu...@gmail.com --- kernel/trace/ftrace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH 2/2] ftrace: Make if condition correctly due to the operator order

2015-07-26 Thread yalin wang
On Jul 26, 2015, at 23:55, Minfei Huang mnfhu...@gmail.com wrote: The if condition will be always true, since the operator has the high priority than operator ||. Use () to quote them to make the if condition correctly. Signed-off-by: Minfei Huang mnfhu...@gmail.com ---

Re: [PATCH 2/2] ftrace: Make if condition correctly due to the operator order

2015-07-26 Thread Minfei Huang
On 07/27/15 at 10:20am, yalin wang wrote: On Jul 26, 2015, at 23:55, Minfei Huang mnfhu...@gmail.com wrote: The if condition will be always true, since the operator has the high priority than operator ||. Use () to quote them to make the if condition correctly. Signed-off-by: