Re: jsm_tty: Deletion of a null pointer check in two functions?

2017-12-18 Thread SF Markus Elfring
> So, what do you prefer? Send it yourself, I became concerned that “my patch” would not get picked up because of Greg's communication settings (for my initial messages when they do not get resent by other contributors). > or want me to send it with your sign-off too? (since was your idea). I

Re: jsm_tty: Deletion of a null pointer check in two functions?

2017-12-18 Thread SF Markus Elfring
> So, what do you prefer? Send it yourself, I became concerned that “my patch” would not get picked up because of Greg's communication settings (for my initial messages when they do not get resent by other contributors). > or want me to send it with your sign-off too? (since was your idea). I

Re: jsm_tty: Deletion of a null pointer check in two functions?

2017-12-18 Thread Guilherme G. Piccoli
On 12/16/2017 04:27 AM, SF Markus Elfring wrote: >> Thanks for the fix. > > Thanks for your positive feedback. > > >> I was on vacation - but now seeing all the analysis made here, > > I assume that special communication settings could trigger > corresponding consequences for the discussed

Re: jsm_tty: Deletion of a null pointer check in two functions?

2017-12-18 Thread Guilherme G. Piccoli
On 12/16/2017 04:27 AM, SF Markus Elfring wrote: >> Thanks for the fix. > > Thanks for your positive feedback. > > >> I was on vacation - but now seeing all the analysis made here, > > I assume that special communication settings could trigger > corresponding consequences for the discussed

Re: jsm_tty: Deletion of a null pointer check in two functions?

2017-12-15 Thread SF Markus Elfring
> Thanks for the fix. Thanks for your positive feedback. > I was on vacation - but now seeing all the analysis made here, I assume that special communication settings could trigger corresponding consequences for the discussed source code adjustment. > if "ch" can't be NULL then please go

Re: jsm_tty: Deletion of a null pointer check in two functions?

2017-12-15 Thread SF Markus Elfring
> Thanks for the fix. Thanks for your positive feedback. > I was on vacation - but now seeing all the analysis made here, I assume that special communication settings could trigger corresponding consequences for the discussed source code adjustment. > if "ch" can't be NULL then please go