Re: [Musicpd-dev-team] The libmpdclient v2.0 project

2008-12-01 Thread Max Kellermann
On 2008/12/01 00:28, Marc Pavot [EMAIL PROTECTED] wrote: 11 other patches available on my repository: Add unit tests to libmpclient No recursive makefiles, please. The patch contains 1 whitespace error. Fix glib include Change include guards Convert MPD_INFO_ENTITY_

Re: [Musicpd-dev-team] The libmpdclient v2.0 project

2008-11-30 Thread Marc Pavot
11 other patches available on my repository: Add unit tests to libmpclient Fix glib include Change include guards Convert MPD_INFO_ENTITY_ to enum Fix currentsong test Cosmetic Improve calls to mpd_executeCommand Small performance improvements

Re: [Musicpd-dev-team] The libmpdclient v2.0 project

2008-11-27 Thread Marc Pavot
- I saw that your idle patch introduced inconsistent indentation (e.g. mpd_glibStartIdle); please configure your editor to indent with tabs. I don't want to start a 'religion war' on this point but don't you think it would be better to use spaces everywhere instead of tabs? This discussion

Re: [Musicpd-dev-team] The libmpdclient v2.0 project

2008-11-27 Thread Max Kellermann
On 2008/11/27 12:10, Marc Pavot [EMAIL PROTECTED] wrote: I don't want to start a 'religion war' on this point but don't you think it would be better to use spaces everywhere instead of tabs? Personally, I prefer spaces, too, but let's just focus on important technical problems. The current

Re: [Musicpd-dev-team] The libmpdclient v2.0 project

2008-11-27 Thread Marc Pavot
Hi, There are some commits on my repository that you may find interesting to integrate. Marc Pavot: Use tabs instead of spaces Split libmpdclient: status Set local includes first for idle.c Split libmpdclient: stats Merge branch 'master' of

Re: [Musicpd-dev-team] The libmpdclient v2.0 project

2008-11-27 Thread Max Kellermann
On 2008/11/28 00:49, Marc Pavot [EMAIL PROTECTED] wrote: There are some commits on my repository that you may find interesting to integrate. Thanks, I've merged all. New disconnect event + IDLE_DISCONNECT = 0x80, This is a clumsy choice, since it will conflict with the next

[Musicpd-dev-team] The libmpdclient v2.0 project

2008-11-26 Thread Max Kellermann
Hi, the libmpdclient project has suffered in the past: instead of having a shared library with a stable API, all clients chose to copy libmpdclient.c into their source directory. That resulted in many forked and different versions of libmpdclient. To fix that, to account for new requirements

Re: [Musicpd-dev-team] The libmpdclient v2.0 project

2008-11-26 Thread Marc Pavot
- portability - even Windows? There's no good C99 compiler on Windows.. mingw32? Just a quick remark about this point: I compile Ario and libmpdclient on windows with mingw32 and it works fine. (I distribute .exe files for Ario).

Re: [Musicpd-dev-team] The libmpdclient v2.0 project

2008-11-26 Thread Marc Pavot
Some of you already promised to help me with that. From now on, I will accept patches (git pull requests preferred). Please keep your git repository rebased on mine. I have created a git repository to help you on this project: http://repo.or.cz/w/libmpdclient/marc.git It is based on your

Re: [Musicpd-dev-team] The libmpdclient v2.0 project

2008-11-26 Thread Max Kellermann
On 2008/11/27 00:33, Marc Pavot [EMAIL PROTECTED] wrote: It is based on your repository and I have done one commit to continue the split of libmpdclient. It creates three new files: connection, idle and return_element. The separation is not perfect (for example some structures and methods in