tokkee.org

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

oops! forgot screen_play.h

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

fixed building on systems with libncursesw  but without <ncursesw/ncurses.h> like Arch Linux

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

translation updates, template and german. another little build fix

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

translation updates, template and german. another little build fix

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

ChangeLog update because of the fixed search screen

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

ChangeLog update because of the fixed search screen

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

search screen works now using the new tag searching api

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

update

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

patch by jat to fix musicpd.org/mantis/view.php?id=1293

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

only compile easy_download when lyrics screen is enabled

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

added qball's easy_download

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

the updated sources

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