Review Request: Fix for SSL handeshake failure in TcpSlaveBase due to server not supporting SSL compression

Thiago Macieira thiago at
Mon Jan 2 22:57:32 GMT 2012

On Monday, 2 de January de 2012 22.18.45, Thomas Baumgart wrote:
> Hi,
> on Monday 02 January 2012 22:08:48 Thiago Macieira wrote:
> > On Monday, 2 de January de 2012 18.53.12, Dawit Alemayehu wrote:
> > > Diff:
> > 
> > 404 - Page does not exist
> > 
> > Reviewboard broken?
> Try

Now I can't find my reviewboard password. It's on the other laptop, which is 
11000 km away.

Anyway, I looked at the change and gave up reviewing it. Dawit, can you make a 
change that doesn't move code around unnecessarily? If you're going to move 
code around, only do that. Then make the other change.

The change has two other problems:
1) it requires new API in kdecore
2) it uses new API in Qt 4.8 that wasn't API reviewed. I want to change it in 
Qt 5.

Thiago Macieira - thiago (AT) - thiago (AT)
   Software Architect - Intel Open Source Technology Center
      PGP/GPG: 0x6EF45358; fingerprint:
      E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <>

More information about the kde-core-devel mailing list