tokkee.org

code style, indent with tabs III

Follow the same code style als MPD itself.

mpdclient: call playlist_clear() in mpdclient_playlist_update()

Instead of freeing the playlist with mpdclient_playlist_free(), call
playlist_clear() to empty the song list.  This fixes a segmentation
fault which occured when you cleared the playlist.

initial draw without delay

Due to a missing refresh() call, ncmpc painted the screen after one
second.  Add refresh(), make ncmpc display immediately.

screen: added struct names

Let us declare struct names in addition to typedef names, so we can
forward-declare them.

use a C99 compiler

lyrics: converted in-process plugins to external programs

In-process plugins are very problematic.  It is much easier and
flexible to move the lyrics plugins to external programs, with a
trivial protocol.  This is work in progress, among the things missing:

 - protocol specification, including exit codes
 - plugin installation
 - plugin search directory
 - run-time configuration (currently hard coded)
 - automatic polling (using glib's main loop?)
 - better and more robust error handling

use GPtrArray instead of GArray for playlist songs

playlist: hide direct accesses in inline functions

Added functions like playlist_length(), playlist_get(),
playlist_replace(), playlist_remove().  Don't access the
mpdclient_playlist struct directly.

mpdclient: added mpdclient_playlist_init(), mpdclient_playlist_clear()

Moved code from mpdclient_new() and mpdclient_playlist_free().  In
mpdclient_disconnect(), call mpdclient_playlist_clear() instead of
mpdclient_playlist_free() (which is now called in mpdclient_free()).

mpdclient: moved code to playlist.c

Move everything which manipulates the mpdclient_playlist struct to
playlist.c.  Many of the functions get a mpdclient pointer instead of
a playlist; this will be changed later.

The functions mpdclient_playlist_update() and
mpdclient_playlist_update_changes() stay in mpdclient.c for now, since
they are tightly connected to the client code.

song: allocate mpd_song from glib slices

Since we compile ncmpc with glib, we can use its slice allocator for
efficient song allocation.  I have added a way to disable this with a
macro.

song: use struct mpd_song

Use a forward declared struct mpd_song instead of the typedef
mpd_Song.

libmpdclient: moved code to song.c

Move everything which manipulates the mpd_Song struct to song.c.
Thousands of lines of too much for libmpdclient.c!

mpdclient: add struct names, for forward declarations

To relax the include dependencies, give all structs real names, in
addition to the typedef names.  This way, other headers can
forward-declare them, and do not need to include mpdclient.h.  In the
future, the typedefs will be removed.

libmpdclient: const pointers

Pass const pointers whenever possible.

code style, indent with tabs II

Follow the same code style als MPD itself.

list_window: update variables without looping

Apparently, somebody was too lazy to derive the formulas for cursor
movement, and wrote lots of while loops.

include cleanup

Include only headers which are really used.

don't use libncursesw if libncurses was selected

No idea what this code was intended for, but if the user chooses
libncurses, ncmpc shouldn't override this manual choice if it finds
libncursesw.

added myself to AUTHORS

set version to 0.12~git

We are on git now, and since we are doing major changes, increase the
version number.

easy_download: fixed the type of write_data()

Fix the type of write_data() to fit curl_write_callback.

use GArray for playlist instead of GList

Using a doubly linked list for the local playlist copy is a huge waste
of both memory and CPU cycles.  Use GArray instead of GList, which is
much faster in this case.

free info entity while iterating

Memory leak: the function mpdclient_playlist_update_changes() did not
free the entity which was returned by mpd_getNextInfoEntity().  Add a
mpd_freeInfoEntity() call at the end of each iteration.

manage tag string allocations in a pool

There are many duplicated strings in the tag database, e.g. many songs
having the same artist.  Don't allocate such strings twice, manage all
strings in a pool.

added support for unix domain sockets

If a host name starts with a slash, it is assumed to be a unix domain
socket path.  The port is ignored.  This code is disabled on WIN32,
until someone tests it.

lyrics: missing "break"

If get_text_line() fails, we should break from the loop.

lyrics: converted global variables to static

The variables "lyrics_text_rows" and "lw" were declared in
src_lyrics.h as static, but they were only actually used in
screen_lyrics.c.  Move both there.

fix unused parameter warnings

Add the "unused" attribute to all function parameters which are indeed
going to be ignored.

imported gcc.h from mpd

gcc.h contains a useful set of macros for setting attributes.

don't declare local variable "options"

In main.c, the global variable "options" is used all over, except in
main(), which contains a shadowing declaration of it.  Remove this
local variable, and use the global "options" instead.  Also don't
pollute conf.c's namespace with the global variable with a CPP hack
(to be removed when we have fixed more of ncmpc's namespace
pollution).

use size_t and unsigned integers

fixed mixed declaration + code

Although they are allowedd in C99, try to avoid them.  Move variable
declarations to the top of the current scope.

fix shadow warnings

Rename local variables and function parameters, so they don't shadow
global variables.

include cleanup

A header should include all headers which he needs.  Move local
includes on top, and let foo.c include foo.h in the first line, to
automatically test its dependencies.

screen_auth() returns result

screen_auth() returns an int, and its result is actually being used.
Only the "return" was missing in its implementation.  Return
_screen_auth()'s result.

fix function prototypes

Add missing prototypes, and fix wrong prototypes.  Convert lots of
functions to "static" when they are only used within the current
source file.

const pointers

Convert pointers to const whenever it is possible.  Fixes all those
-Wconst warnings.

clock: don't initialize static variables with zero

Uninitialized static variables are put into BSS, which is
automatically zeroed.  Initializing doesn't hurt, but in this case,
there were elements missing, emitting a gcc warning.

screen: moved code to paint_top_window2()

Nearly all of the code in paint_top_window() is on indent level 2,
just because it depends on a condition.  Make this more readable and
move to indent level 1 by moving this code to a separate function.

use !NDEBUG instead of DEBUG

NDEBUG is more commonly used than DEBUG, e.g. in assert.h.  Convert
all macros to NDEBUG, and declare NDEBUG when debugging is disabled.

enable gcc warnings

Enable all useful warnings which were also enabled in MPD.  Steal
m4/mpd_check_cflag.m4 from MPD.

configure: updated m4/*.m4

Copied m4 files from /usr/share/aclocal (Debian Lenny).

code style, indent with tabs

Follow the same code style als MPD itself.  This patch only fixes
parts of the code which are going to be touched in the following bunch
of patches, i.e. there will be more "code style" patches in the
future.

remove unused static functions

Also remove some commented code and unused function parameters.

splash: removed the useless splash screen

The code looks strange and spews lots of warnings.  Since it does
nothing useful, and wastes the user's precious time, simply remove it.

improve escaping of URLs

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@7141 09075e82-0dd4-0310-85a5-a0d7c8717e4f

mixed up the files, sorry

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@7140 09075e82-0dd4-0310-85a5-a0d7c8717e4f

slovak translation updated

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@7139 09075e82-0dd4-0310-85a5-a0d7c8717e4f

added Slovak translation

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@7136 09075e82-0dd4-0310-85a5-a0d7c8717e4f

make item highlighiting more appropriate

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@7062 09075e82-0dd4-0310-85a5-a0d7c8717e4f

cmd_select_all added

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@7061 09075e82-0dd4-0310-85a5-a0d7c8717e4f

fixed 0001496

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@6817 09075e82-0dd4-0310-85a5-a0d7c8717e4f

Removed nested functions

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@6811 09075e82-0dd4-0310-85a5-a0d7c8717e4f

2 critical bugs in leoslyrics lyrics src fixed

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@6630 09075e82-0dd4-0310-85a5-a0d7c8717e4f

fix segfault

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5773 09075e82-0dd4-0310-85a5-a0d7c8717e4f

updated configure.ac

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5730 09075e82-0dd4-0310-85a5-a0d7c8717e4f

ugh!

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5621 09075e82-0dd4-0310-85a5-a0d7c8717e4f

zpdated autogen.sh

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5619 09075e82-0dd4-0310-85a5-a0d7c8717e4f

new autogen.sh

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5561 09075e82-0dd4-0310-85a5-a0d7c8717e4f

dirty hack for archlinux

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5546 09075e82-0dd4-0310-85a5-a0d7c8717e4f

way too much stuff to describe here

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5346 09075e82-0dd4-0310-85a5-a0d7c8717e4f

ChangeLog update

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5315 09075e82-0dd4-0310-85a5-a0d7c8717e4f

lyrics updates, leoslyrics fixes

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5314 09075e82-0dd4-0310-85a5-a0d7c8717e4f

fix user plugin dir; code cleanup

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5220 09075e82-0dd4-0310-85a5-a0d7c8717e4f

fix error display on lyrics screen

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5219 09075e82-0dd4-0310-85a5-a0d7c8717e4f

forgot a Makefile.in

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5218 09075e82-0dd4-0310-85a5-a0d7c8717e4f

added a plugin system for lyrics sources

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5217 09075e82-0dd4-0310-85a5-a0d7c8717e4f

make buildsystem GNU standard

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5216 09075e82-0dd4-0310-85a5-a0d7c8717e4f

Daniel rocked the house

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5041 09075e82-0dd4-0310-85a5-a0d7c8717e4f

go to root/parent dir key added

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5034 09075e82-0dd4-0310-85a5-a0d7c8717e4f

spanish translation updated

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@5026 09075e82-0dd4-0310-85a5-a0d7c8717e4f

build problems fixed

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4996 09075e82-0dd4-0310-85a5-a0d7c8717e4f

filelist sorting improved

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4953 09075e82-0dd4-0310-85a5-a0d7c8717e4f

updated galician translation

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4931 09075e82-0dd4-0310-85a5-a0d7c8717e4f

you can specify %shortalbum% for playlist song markup, parent directory is shown in the browse screen title

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4871 09075e82-0dd4-0310-85a5-a0d7c8717e4f

http://musicpd.org/mantis/view.php?id=1326

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4758 09075e82-0dd4-0310-85a5-a0d7c8717e4f

http://musicpd.org/mantis/view.php?id=1324

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4757 09075e82-0dd4-0310-85a5-a0d7c8717e4f

multi liner ;) fixed segfault

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4749 09075e82-0dd4-0310-85a5-a0d7c8717e4f

disable splash by default

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4744 09075e82-0dd4-0310-85a5-a0d7c8717e4f

update manpage

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4743 09075e82-0dd4-0310-85a5-a0d7c8717e4f

oppps forgot 2 files...

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4742 09075e82-0dd4-0310-85a5-a0d7c8717e4f

due to bensonk's demand i added a splash screen.

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4741 09075e82-0dd4-0310-85a5-a0d7c8717e4f

use older revision till serious bugfix

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4704 09075e82-0dd4-0310-85a5-a0d7c8717e4f

make compilation work again

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4697 09075e82-0dd4-0310-85a5-a0d7c8717e4f

code cleanup

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4695 09075e82-0dd4-0310-85a5-a0d7c8717e4f

fix a bug the makes utf8 names behave evil

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4683 09075e82-0dd4-0310-85a5-a0d7c8717e4f

spelling fixes by avuton

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4615 09075e82-0dd4-0310-85a5-a0d7c8717e4f

fix a bug that doubled empty lines when loading lyrics from hd

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4601 09075e82-0dd4-0310-85a5-a0d7c8717e4f

avuton fixed some warnings

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4600 09075e82-0dd4-0310-85a5-a0d7c8717e4f

fix a bug when writing lyrics. 1st line is written now

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4588 09075e82-0dd4-0310-85a5-a0d7c8717e4f

now really show the save lyrics command on the help screen

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4587 09075e82-0dd4-0310-85a5-a0d7c8717e4f

show lyrics saving command on help screen + .pot and german po update

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4581 09075e82-0dd4-0310-85a5-a0d7c8717e4f

delte debug msg

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4580 09075e82-0dd4-0310-85a5-a0d7c8717e4f

it's august?

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4579 09075e82-0dd4-0310-85a5-a0d7c8717e4f

important compile fix!

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4578 09075e82-0dd4-0310-85a5-a0d7c8717e4f

lots of cool stuff!

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4577 09075e82-0dd4-0310-85a5-a0d7c8717e4f

fixed up lyrics loading interruption

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4576 09075e82-0dd4-0310-85a5-a0d7c8717e4f

fix bug in lyrics screen

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4572 09075e82-0dd4-0310-85a5-a0d7c8717e4f

interruption of lyrics loading + po blabla

git-svn-id: https://svn.musicpd.org/ncmpc/branches/tradiaz@4567 09075e82-0dd4-0310-85a5-a0d7c8717e4f