[04:09] *** xecutercmd has joined #linux-media
[04:15] *** xecutertool has quit IRC (Ping timeout: 480 seconds)
[05:55] *** mvaittin has joined #linux-media
[06:20] *** Mo has joined #linux-media
[06:40] *** ten157237743246305066182150355 has joined #linux-media
[06:42] *** ten15723774324630506618215035 has quit IRC (Remote host closed the connection)
[06:48] *** ten157237743246305066182150355 has quit IRC (Ping timeout: 480 seconds)
[06:54] *** Mo_ has joined #linux-media
[07:00] *** Mo has quit IRC (Ping timeout: 480 seconds)
[07:21] *** dcz_ has joined #linux-media
[07:24] *** djrscally has joined #linux-media
[07:25] *** djrscally has quit IRC ()
[07:26] *** djrscally has joined #linux-media
[07:54] *** Mo-jorisin has joined #linux-media
[08:00] *** Mo_ has quit IRC (Ping timeout: 480 seconds)
[08:23] *** dcz_ has quit IRC (Quit: Konversation terminated!)
[08:28] *** ao2 has joined #linux-media
[08:32] *** jmassot has quit IRC (Remote host closed the connection)
[08:43] *** jmassot has joined #linux-media
[08:54] *** Mo_ has joined #linux-media
[09:00] *** Mo-jorisin has quit IRC (Ping timeout: 480 seconds)
[09:03] *** hansg has joined #linux-media
[09:19] *** sgruszka has joined #linux-media
[09:28] <ribalda> metaphisical question. Reviewed-by does also imply Acked-by?
[09:37] <hverkuil> yes
[09:41] <pinchartl> I'd say it depends
[09:41] <pinchartl> some people consider that Rb is stronger than Ab, other people think the other way around
[09:44] <kbingham> To me: RB means "I've checked this code, and see no bugs, and I'm happy for it to be merged" ... AB: means "I haven't specifically reviewed this - but I believe it should be merged for $WHATEVER_REASONS"
[09:45] <javierm> kbingham: that's who I understand it too and in line with what's documented in Documentation/process/submitting-patches.rst
[09:46] <pinchartl> that's how I interpret it, but I know that for some people Rb means "I've reviewed the patch and it looks fine to me", while Ab means "I put my reputation at stake and say we should merge this"
[09:47] <javierm> pinchartl: yeah and at least what I usually do is to use Ab if the changes is for code I maintain and use Rb when I reviewed the patch but is not for code I maintain 
[09:47] <javierm> that is, use Ab to say "I think is OK to merge this patch" 
[09:48] <pinchartl> I use Ab to say "no objection but I'm no expert on this", and Rb to give my real stamp of full approval
[09:48] <javierm> pinchartl: I see
[09:49] <javierm> pinchartl: so for you Rb implies Ab 
[09:49] <pinchartl> for me yes
[09:49] <javierm> but not the other way around 
[09:50] <javierm> I think the problem is that Ab doesn't tell how much the person doing the ack reviewed the code 
[09:51] <pinchartl> Rb certainly implies the person has reviewed the patch, yes
[09:51] <javierm> pinchartl: exactly and then would imply an ack 
[09:52] <javierm> I always use it as a if ack implied rb but I see now that this may not be the case 
[09:53] <javierm> s/now// actually since I knew it but never gave that much thought 
[09:54] *** Mo-jorisin has joined #linux-media
[10:00] *** Mo_ has quit IRC (Ping timeout: 480 seconds)
[10:04] *** mehdi-djait3 has joined #linux-media
[10:30] *** mripard has joined #linux-media
[10:44] <hverkuil> I interpret r-b and a-b the same way pinchartl does. r-b is in-depth, a-b is 'whatever, I'm OK with it', or possibly 'I quickly scanned it and it looks OK to me'.
[10:46] *** mehdi-djait33 has joined #linux-media
[10:48] <javierm> hverkuil, pinchartl: thanks for your insights, very useful 
[10:50] *** ten157237743246305066182150355 has joined #linux-media
[10:51] *** mehdi-djait3 has quit IRC (Ping timeout: 480 seconds)
[10:54] *** Mo_ has joined #linux-media
[11:00] *** Mo-jorisin has quit IRC (Ping timeout: 480 seconds)
[11:30] *** danitool has joined #linux-media
[11:54] *** Mo-jorisin has joined #linux-media
[12:00] *** Mo_ has quit IRC (Ping timeout: 480 seconds)
[12:52] *** mvaittin has quit IRC (Ping timeout: 480 seconds)
[12:54] *** Mo_ has joined #linux-media
[13:00] *** Mo-jorisin has quit IRC (Ping timeout: 480 seconds)
[13:54] *** Mo-jorisin has joined #linux-media
[14:00] *** Mo_ has quit IRC (Ping timeout: 480 seconds)
[14:10] *** mehdi-djait33 has quit IRC ()
[14:12] *** mehdi-djait33 has joined #linux-media
[14:18] *** charrus has quit IRC (Quit: https://quassel-irc.org - Chat comfortably. Anywhere.)
[14:24] *** wallacer has joined #linux-media
[14:25] *** charrus has joined #linux-media
[14:37] *** Schrostfutz has joined #linux-media
[14:37] *** nickware has quit IRC (Read error: Connection reset by peer)
[14:40] *** nickware has joined #linux-media
[14:58] *** Mo-jorisin has quit IRC (Ping timeout: 480 seconds)
[14:58] *** sgruszka has quit IRC (Quit: Leaving)
[15:05] *** charrus has quit IRC (Quit: https://quassel-irc.org - Chat comfortably. Anywhere.)
[15:05] *** charrus has joined #linux-media
[15:10] *** eelstrebor has joined #linux-media
[15:11] *** mvaittin has joined #linux-media
[15:12] *** hansg has quit IRC (Remote host closed the connection)
[15:12] *** danitool has quit IRC (Quit: Cubum autem in duos cubos, aut quadratoquadratum in duos quadratoquadratos)
[15:18] *** mvaittin_ has joined #linux-media
[15:18] *** mvaittin has quit IRC (Read error: Connection reset by peer)
[15:38] *** Schrostfutz has quit IRC (Remote host closed the connection)
[15:38] *** Schrostfutz has joined #linux-media
[15:52] *** Schrostfutz has quit IRC (Ping timeout: 480 seconds)
[16:06] *** mvaittin_ has quit IRC (Ping timeout: 480 seconds)
[16:17] *** Schrostfutz has joined #linux-media
[16:25] *** mvaittin_ has joined #linux-media
[16:38] *** charrus has quit IRC (Quit: https://quassel-irc.org - Chat comfortably. Anywhere.)
[16:43] *** charrus has joined #linux-media
[17:02] *** danitool has joined #linux-media
[17:17] *** Schrostfutz has quit IRC (Ping timeout: 480 seconds)
[17:23] *** eelstrebor has quit IRC (Quit: Ex-Chat)
[17:24] *** eelstrebor has joined #linux-media
[17:27] *** mvaittin_ has quit IRC (Ping timeout: 480 seconds)
[17:39] *** mvaittin_ has joined #linux-media
[17:40] *** Schrostfutz has joined #linux-media
[17:53] *** BrianG61UK has quit IRC (Read error: Connection reset by peer)
[18:25] *** mehdi-djait33 has quit IRC ()
[18:44] *** mrpops2ko has quit IRC ()
[18:51] *** ten157237743246305066182150355 has quit IRC (Remote host closed the connection)
[18:51] *** ten157237743246305066182150355 has joined #linux-media
[19:00] *** mrpops2ko has joined #linux-media
[19:31] *** dcz_ has joined #linux-media
[19:31] *** mrpops2ko has quit IRC ()
[19:38] *** mrpops2ko has joined #linux-media
[19:46] *** epoll has quit IRC (Ping timeout: 480 seconds)
[19:48] *** gouchi has joined #linux-media
[19:56] *** epoll has joined #linux-media
[20:03] *** mvaittin_ has quit IRC (Ping timeout: 480 seconds)
[20:07] *** ao2 has quit IRC (Quit: Leaving)
[20:43] *** eelstrebor has quit IRC (Quit: Ex-Chat)
[20:50] *** dcz_ has quit IRC (Quit: Konversation terminated!)
[21:02] *** Schrostfutz has quit IRC (Ping timeout: 480 seconds)
[21:44] *** ten157237743246305066182150355 has quit IRC (Remote host closed the connection)
[21:45] *** ten157237743246305066182150355 has joined #linux-media
[21:50] *** gouchi has quit IRC (Remote host closed the connection)
[22:34] *** BrianG61UK has joined #linux-media
[22:43] *** wallacer has quit IRC (Remote host closed the connection)
[22:44] *** wallacer has joined #linux-media