Re: [Lxde-list] LXPanel Weather Plugin

2015-07-13 Thread Piotr Sipika
Hey Andriy, 13 lip 2015 06:39 "Andrej N. Gritsenko" napisaƂ(a): > The commit where many different changes are all-in-one is > very hard to be understood, to review ang get commented, and your biggest > commit has at least 5 things in it: a) indentations changes; b) many data > members renamed; c)

Re: [Lxde-list] LXPanel Weather Plugin

2015-07-13 Thread Andrej N. Gritsenko
Hello! Piotr Sipika has written on Saturday, 11 July, at 22:46: >On 08/01/2014 08:33 PM, Andrej N. Gritsenko wrote: >> 4) might it be better to use conditional compilation instead of using >> conditional execution? That way we can omit some unused code when it's >> used as panel plugin. >It l

Re: [Lxde-list] LXPanel Weather Plugin

2015-07-11 Thread Piotr Sipika
On 08/01/2014 08:33 PM, Andrej N. Gritsenko wrote: > Hello! Hi Andriy, All, > No updates from you, though I found some updates in your repository. It's been a while and I've made some more updates, details follow: > 1) the function gtk_weather_button_pressed() will be never called for >

Re: [Lxde-list] LXPanel Weather Plugin

2014-08-04 Thread Piotr Sipika
On 08/01/2014 08:33 PM, Andrej N. Gritsenko wrote: > Hello! Hi! > No updates from you, though I found some updates in your repository. Yup, I try to work on the plugin when I have some time, but I don't want to spam the list with updates for incomplete functionality... > Few notes on it:

Re: [Lxde-list] LXPanel Weather Plugin

2014-08-01 Thread Andrej N. Gritsenko
Hello! Piotr Sipika has written on Sunday, 6 July, at 19:05: >I'll put in some time here and there, and will provide status, hopefully >on a weekly basis. No updates from you, though I found some updates in your repository. Few notes on it: 1) the function gtk_weather_button_pressed() w

Re: [Lxde-list] LXPanel Weather Plugin

2014-07-07 Thread Andrej N. Gritsenko
Hello Piotr! Piotr Sipika has written on Sunday, 6 July, at 19:05: >I'll put in some time here and there, and will provide status, hopefully >on a weekly basis. That's great! Thank you very much! >Warmest wishes! >Piotr With best wishes, Andriy. ---

Re: [Lxde-list] LXPanel Weather Plugin

2014-07-06 Thread Piotr Sipika
On 07/05/2014 06:05 AM, Andrej N. Gritsenko wrote: > Hello! Hi Andriy! >> I don't want to hold back the release, so my plan is to do the following >> by the end of this weekend: >> 1) fix configuration persistence Still working on this. Turns out that with the migration to the new plugin API,

Re: [Lxde-list] LXPanel Weather Plugin

2014-07-05 Thread Piotr Sipika
> The > pointer to config for the plugin instance is given as a parameter on the > instance creation so it may be remembered in the private data to use for > that purpose. Ah yes! Thank you! > Please, inspect LXPanelPluginInit description in the plugin.h header > file - the config callback sh

Re: [Lxde-list] LXPanel Weather Plugin

2014-07-05 Thread Andrej N. Gritsenko
Hello! Piotr Sipika has written on Friday, 4 July, at 20:43: >On 07/03/2014 04:22 PM, Andrej N. Gritsenko wrote: >> What is the current state of this work? I believe it is the time to >> make a release for LXPanel so we need some progress into this plugin as >> well. >I'm 95% done migrat

Re: [Lxde-list] LXPanel Weather Plugin

2014-07-04 Thread Piotr Sipika
On 07/03/2014 04:22 PM, Andrej N. Gritsenko wrote: > Hello! Hey Andriy! > What is the current state of this work? I believe it is the time to > make a release for LXPanel so we need some progress into this plugin as > well. I'm 95% done migrating the plugin to the new API. The only thing m

Re: [Lxde-list] LXPanel Weather Plugin

2014-07-03 Thread Andrej N. Gritsenko
Hello! Piotr Sipika has written on Sunday, 30 March, at 21:27: >I took a bit of time this week and had a look as to what still needs to >be done. >I've started on cleanup and minor usability updates and will continue to >make those changes as I go through the debugger/valgrind. What is t

Re: [Lxde-list] LXPanel Weather Plugin

2014-03-31 Thread Andrej N. Gritsenko
Hello! Piotr Sipika has written on Sunday, 30 March, at 21:27: >I took a bit of time this week and had a look as to what still needs to >be done. Great news! Thank you very much! >I've started on cleanup and minor usability updates and will continue to >make those changes as I go through

Re: [Lxde-list] LXPanel Weather Plugin

2014-03-30 Thread gary sheppard
Looking forward to this! On Sun, Mar 30, 2014 at 6:27 PM, Piotr Sipika wrote: > On 03/25/2014 10:05 PM, Andrej N. Gritsenko wrote: > > Hello! > Hey Andriy. > > > How does the progress going on that? > I took a bit of time this week and had a look as to what still needs to > be done. > > I've

Re: [Lxde-list] LXPanel Weather Plugin

2014-03-30 Thread Piotr Sipika
On 03/25/2014 10:05 PM, Andrej N. Gritsenko wrote: > Hello! Hey Andriy. > How does the progress going on that? I took a bit of time this week and had a look as to what still needs to be done. I've started on cleanup and minor usability updates and will continue to make those changes as I go t

Re: [Lxde-list] LXPanel Weather Plugin

2014-03-26 Thread Piotr Sipika
On 03/25/2014 10:05 PM, Andrej N. Gritsenko wrote: > How does the progress going on that? Slowly... Work is keeping me busier than normal. I'll provide more details this weekend. Best wishes! Piotr -- Learn Graph Databas

Re: [Lxde-list] LXPanel Weather Plugin

2014-03-25 Thread Andrej N. Gritsenko
Hello! Piotr Sipika has written on Tuesday, 18 February, at 11:01: >I keep on trying to get going on the fixes and now that the plugin made >it to master, I better get my stuff together... >I'll get working on Daniele's suggestions, first (mostly cleanup and >minor functionality tweaks). Then

Re: [Lxde-list] LXPanel Weather Plugin

2014-02-19 Thread Andrej N. Gritsenko
Hello! Piotr Sipika has written on Tuesday, 18 February, at 11:01: >> 1) migrate it to new plugins API (feel free to ask me details). >> 2) use lightweight FmXmlFile API instead of heavy libxml one. After closer look into your plugin sources I should admit using the libxml2 may be unavoi

Re: [Lxde-list] LXPanel Weather Plugin

2014-02-18 Thread Piotr Sipika
> Hello! Hi Andriy, > Since lxpanel is GLib/GTK application I would suggest to use GThread and > GCancellable APIs, better don't call pthreads directly. And mentioned > pthread_kill() is not reliable operation which may cause problems in some > cases, avoid it, please. Yes, I'll definitely do

Re: [Lxde-list] LXPanel Weather Plugin

2014-02-17 Thread Andrej N. Gritsenko
Hello! Piotr Sipika has written on Thursday, 14 February, at 11:44: >Another quick update on the plugin: >I'm slowly reworking the logic to get the plugin up without slowing >down the panel. >In the process I'm also trying to incorporate fixes suggested by >Daniele and Globe Trotter. >This s

Re: [Lxde-list] LXPanel Weather Plugin

2013-02-14 Thread Piotr Sipika
Another quick update on the plugin: I'm slowly reworking the logic to get the plugin up without slowing down the panel. In the process I'm also trying to incorporate fixes suggested by Daniele and Globe Trotter. This statement: > (pthread_kill has been removed in newer versions of the pthread/lib

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-21 Thread Globe Trotter
Thanks, Piotr! I am looking forward to using this, in particular as a standalone application in linux. --- On Mon, 1/21/13, Piotr Sipika wrote: > From: Piotr Sipika > Subject: Re: [Lxde-list] LXPanel Weather Plugin > To: lxde-list@lists.sourceforge.net > Date: Monday, January 21,

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-21 Thread Piotr Sipika
Hi All, Just a quick update on the plugin: I updated the code to take care of warnings as submitted by Daniele Forsi [1]. This was very useful as I got to revisit (and fix) some of the logic (available at github). The thing that will probably take a bit of time is the forecast retrieval multi-th

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-20 Thread Globe Trotter
hout manual editing to the files so that it is >included for perpetuity? Many thanks again for this plug-in. I am especially happy that you have chosen to make it as a standalone version (if needed). --- On Wed, 1/16/13, Piotr Sipika wrote: > From: Piotr Sipika > Subject: Re: [Lxd

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-16 Thread Piotr Sipika
Henry, > I think the forecast retrieval always needs to be done either in a > separate thread or with non-blocking calls (if that works -- I don't > know how network programming really works, maybe there is another way). > We don't want the panel to freeze for a few seconds just because the > netw

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-16 Thread Henry Gebhardt
On Wed, Jan 16, 2013 at 03:47:10PM -0500, Piotr Sipika wrote: > > Another thing I forgot to mention: On startup the plugin takes a few > > seconds. I assume it is waiting for data? This shouldn't happen. Fast > > startup is essential, imho. > > If this happens once the location is set (please let

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-16 Thread Piotr Sipika
Daniele, > I like it and I would be happy to provide the Italian translation > > you may want to check the following error messages from cppcheck (to > see them just run cppcheck src): > and the warnings from clang in the attached clang.txt file (the > configure step used gcc, but then I ran:

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-16 Thread Daniele Forsi
2013/1/15 Piotr Sipika: > I would appreciate the community's feedback and would like to ask for > the plugin to be included in mainline LXPanel source. I like it and I would be happy to provide the Italian translation you may want to check the following error messages from cppcheck (to see them

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-16 Thread Piotr Sipika
Henry, All, > Another thing I forgot to mention: On startup the plugin takes a few > seconds. I assume it is waiting for data? This shouldn't happen. Fast > startup is essential, imho. If this happens once the location is set (please let me know whether that's the case), then yes, it is most-like

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-16 Thread Piotr Sipika
Henry, All, > Would be great if you could use panel_draw_label_text() (defined in > src/panel.h) for the text on the panel. That way it will make use of the > color and font size set globally in the panel configuration. I can try to work it into the source, but the main idea is that the custom wi

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-16 Thread Piotr Sipika
Andriy, All, >> While the applet/plugin uses the LXPanel configuration file and layout, >> the application uses a glib-style key-value group file (created in >> $HOME/.config/lxweather/ and named 'config'). > > Probably that's appropriate. Though may be it would be better to have > it in the

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-15 Thread Henry Gebhardt
On Tue, Jan 15, 2013 at 11:22:51PM -0500, Henry Gebhardt wrote: > On Wed, Jan 16, 2013 at 01:44:48AM +0200, Andrej N. Gritsenko wrote: > > Piotr Sipika has written on Tuesday, 15 January, at 16:51: > > >This email will be rather long - it pertains to the weather plugin for > > >LXPanel. > > > > >A

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-15 Thread Henry Gebhardt
On Wed, Jan 16, 2013 at 01:44:48AM +0200, Andrej N. Gritsenko wrote: > Piotr Sipika has written on Tuesday, 15 January, at 16:51: > >This email will be rather long - it pertains to the weather plugin for > >LXPanel. > > >A bit of a summary: > > >I put together a custom widget which shows the curr

Re: [Lxde-list] LXPanel Weather Plugin

2013-01-15 Thread Andrej N. Gritsenko
Hello! Piotr Sipika has written on Tuesday, 15 January, at 16:51: >This email will be rather long - it pertains to the weather plugin for >LXPanel. >A bit of a summary: >I put together a custom widget which shows the current weather >conditions for a configured location. The functionality ha