<html>

 <body>

  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">

   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">

    <tr>

     <td>

      This is an automatically generated e-mail. To reply, visit:

      <a href="http://git.reviewboard.kde.org/r/108029/">http://git.reviewboard.kde.org/r/108029/</a>

     </td>

    </tr>

   </table>

   <br />

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Added a bug to fix this:

https://bugs.kde.org/show_bug.cgi?id=314781

Gonna close review.</pre>

 <br />

<p>- Àlex</p>

<br />

<p>On December 30th, 2012, 4:57 p.m. UTC, Lukáš Karas wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">

 <tr>

  <td>

<div>Review request for Solid, Àlex Fiestas and Dan Vrátil.</div>

<div>By Lukáš Karas.</div>

<p style="color: grey;"><i>Updated Dec. 30, 2012, 4:57 p.m.</i></p>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>

 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">

 <tr>

  <td>

   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">It something fails and your list of outputs is empty, "embeddedOutput" will be NULL. It is better check this situation before use "embeddedOutput".

For example if you running KDED in VNC server with RandR 1.1, you get something like this:

RandR version 1.1 too old

...

X Error: BadRequest (invalid request code or no such operation) 1

  Extension:    149 (RANDR)

  Minor opcode: 6 (RRGetScreenSizeRange)

  Resource id:  0x2e

and emtpy list of outputs...</pre>

  </td>

 </tr>

</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>

<ul style="margin-left: 3em; padding-left: 0;">

 <li>kded/generator.cpp <span style="color: grey">(4d57571)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/108029/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>