On Sun, Oct 28, 2012 at 8:03 PM, Kevin Krammer <span dir="ltr"><<a href="mailto:krammer@kde.org" target="_blank">krammer@kde.org</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br></div><div class="im">
> This is for the parsing purposes - the library uses QJson parser/mapper,<br>
> which automagically maps the received json data to qobjects, otherwise<br>
> there would have to be manual parsing everywhere (and the facebook jsons<br>
> are huge), which means more code, more error possibilities, more<br>
> maintaining requirement and worse readability (compared to two lines QJson<br>
> mapper). So I'd like to leave this one as is.<br>
<br>
</div>I haven't had a look at the QJson library internals (yet), but from its usage<br>
it looks like that it is only using instances of those QObject classes to<br>
provide a convenient mapper of map keys to conversion functions (the property<br>
setters).<br>
<br>
This would make them an internal implementation detail, something more<br>
convenient than manually writing a mapping of string to function pointer but<br>
also just private.<br>
<br>
As I said I'll have a look into QJson, but unless I am gravely mistaken it<br>
only needs such QObjects as a generic accessor API, not as the actual data<br>
object.<br></blockquote><div><br></div><div>Thanks. I fixed all the issues you pointed out except this one.</div><div><br></div><div>Also I checked for the naming and here's what I found [1]:</div><div><br></div><div>

  6. You may not combine our Brand Assets, or elements of our Brand Assets, with your own name or mark or generic terms.</div><div><br></div><div>So if "lib" is a generic term, "k" is sort of our mark, I guess "libkfacebook" is pretty much off limits, right? Either way, we might be just better off changing the name and rest calm. I'm thinking going in steps of "libkgapi" - "libkfbapi". Thoughts?</div>

<div><br></div><div>[1] - <a href="https://www.facebook.com/brandpermissions/logos.php">https://www.facebook.com/brandpermissions/logos.php</a></div><div><br></div><div>Cheers</div></div>-- <br><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div>

<br>