[kgraphviewer-devel] KGraphViewer lib

[Sandro Andrade]

On Thu, May 27, 2010 at 2:23 PM,  <kleag at free.fr> wrote:
> Oh yes, sorry I forgot that ! So, we should readd a slot to pass a graph_t*... I'll do it as soon as possible except if you want to take that in charge.

I can do it :)

Thanks,
Sandro

>
> Gaël
> ----- Mail Original -----
> De: "Sandro Andrade" <sandroandrade at kde.org>
> À: kgraphviewer-devel at kde.org
> Envoyé: Jeudi 27 Mai 2010 19h12:58 GMT +01:00 Amsterdam / Berlin / Berne / Rome / Stockholm / Vienne
> Objet: Re: [kgraphviewer-devel] KGraphViewer lib
>
> On Thu, May 27, 2010 at 2:03 PM,  <kleag at free.fr> wrote:
>> Hi Sandro,
>
> Hi Gael :)
>
>>
>> I'm very sorry to again answer so lately. I was at a conference last week with few time for mails and moreover, my kmail was blocked: it has created thousands of empty messages that have poluted my IMAP account :-(
>
> Fine, no problems :)
>
>>
>> There is no more loadLibrary because the loading through external process or library is now handled through an option and then you can use the openUrl slot for both methods, eventually after setting you prefered method using setLayoutMethod (in the interface). And in fact, the default is now to use the library...
>
> Yes, you are right but the question is: regardless the layout method
> used, graph is loaded from file by using localFilePath().
> That was my first implementation in KDevelop plugin: generate a
> temporary file and pass it to KGraphviewer kpart.
> We changed it later to accept a graph_t * directly to remove the need
> for generating temporary files all time.
>
> Now, I can't provide the graph to kpart without generating a file and
> that would imply in performance degradation since
> graphs are usually generated quit often.
>
> Please let know if I am getting things incorrectly :)
>
> Thanks,
> Sandro
>
>>
>> So, you still don't have to link with the lib. In fact kgraphviewer uses this method and it is not linked.
>>
>>
>> I think that the API is better this way.
>>
>> Cheers,
>>
>> Gaël
>> ----- Mail Original -----
>> De: "Sandro Andrade" <sandroandrade at kde.org>
>> À: kgraphviewer-devel at kde.org
>> Envoyé: Samedi 22 Mai 2010 17h43:49 GMT +01:00 Amsterdam / Berlin / Berne / Rome / Stockholm / Vienne
>> Objet: Re: [kgraphviewer-devel] KGraphViewer lib
>>
>> Hi Gael,
>>
>> I've noticed that slotLoadLibrary(graph_t *) was removed from kpart.
>> loadLibrary(graph_t *) is still in dotgraphview but there is no way to
>> use it from kpart apart from calling
>> kpart->widget()->loadLibrary(graph *) directly. But that would require
>> linking to kgraphviewer instead of using it only as a kpart.
>>
>> What would be the right approach to have this feature back ? I'm not
>> sure if kgraphviewer_interface is the best place to have this. What
>> were the reasons to remove this slot ?
>>
>> See you,
>> Sandro
>>
>> 2010/5/13 Gaël (aka Kleag) <kleag at free.fr>:
>>> Hello,
>>>
>>> Even if it is not completely finished, I finally commited yesterday evening the
>>> refactoring separating the kgraphviewerlib library and the kgraphviwer_part.
>>> It works in kgraphviewer but kgrapheditor compiles but don't works.
>>>
>>> I commited because I miss time to clean it and that gives you the opportunity
>>> to help in designing the right API for both the part and the lib. Now, it
>>> should be decided what in the lib API should become a slot for example, as the
>>> widget can now be directly integrated and thus need to react with signals and
>>> slots.
>>>
>>> Gaël
>>> --
>>> KsirK - a world domination strategy game
>>> http://techbase.kde.org/Projects/Games/Tactic_and_Strategy/KsirK
>>>
>>> KGraphViewer - a GraphViz dot graphs viewer and editor based on a reusable
>>> part
>>> http://extragear.kde.org/apps/kgraphviewer
>>> _______________________________________________
>>> kgraphviewer-devel mailing list
>>> kgraphviewer-devel at kde.org
>>> https://mail.kde.org/mailman/listinfo/kgraphviewer-devel
>>>
>> _______________________________________________
>> kgraphviewer-devel mailing list
>> kgraphviewer-devel at kde.org
>> https://mail.kde.org/mailman/listinfo/kgraphviewer-devel
>> _______________________________________________
>> kgraphviewer-devel mailing list
>> kgraphviewer-devel at kde.org
>> https://mail.kde.org/mailman/listinfo/kgraphviewer-devel
>>
> _______________________________________________
> kgraphviewer-devel mailing list
> kgraphviewer-devel at kde.org
> https://mail.kde.org/mailman/listinfo/kgraphviewer-devel
> _______________________________________________
> kgraphviewer-devel mailing list
> kgraphviewer-devel at kde.org
> https://mail.kde.org/mailman/listinfo/kgraphviewer-devel
>