Adding if_up support for FreeBSD.

Moved interface_up(...) from linux.{c.h} to common.{c,h} and taught it
to check for ENXIO as well to make it work on FreeBSD.

Signed-off-by: Nikos Ntarmos <ntarmos@cs.uoi.gr>

Bringing FreeBSD up to date with latest commits

Several things were broken after the switch to 1.7-rc branches. This
diff fixes all compilations issues and updates some functions that were
left empty previously.

Signed-off-by: Nikos Ntarmos <ntarmos@cs.uoi.gr>

sysfs is a Linux-only feature.

Moving sysfs-related functions and defs out of common.h and into
linux.h, as sysfs exists only on Linux, and updating openbsd.c,
netbsd.c, and freebsd.c accordingly.

Signed-off-by: Nikos Ntarmos <ntarmos@cs.uoi.gr>

Extra includes to fix compiling on FreeBSD

Signed-off-by: Nikos Ntarmos <ntarmos@cs.uoi.gr>

Fix FOPENCOOKIE detection glitch

Using AM_CONDITIONAL's inside an if block is considered "confusing" by
the automake manual[1] and in fact leads to configure code that won't
work unless fopencookie actually exists in a system.

[1] http://sources.redhat.com/automake/automake.html#Conditionals

Signed-off-by: Nikos Ntarmos <ntarmos@cs.uoi.gr>

Fix a typo and teach linux code to use data

Typo: __LINUX__ was used in a conditional code block, while the rest of
      the code uses __linux__

data: free_text_objects(...) #define's 'data' to be 'obj->data' to make
  the code more compact. The linux-specific parts of the code still
  used obj->data, which should expand to obj->obj->data, which is
  wrong.

Signed-off-by: Nikos Ntarmos <ntarmos@cs.uoi.gr>

update README and manpage

add eval object, evaluating it's args

This object makes use of the possibility to escape dollar signs in TEXT.
Take the following example in the conkyrc:
| $${downspeed ${gw_iface}}
will be evaluated to (assuming the gw_iface is eth0):
| ${downspeed eth0}
and finally interpreted to print the gateway interface's downspeed rate.

fix dollar escape in TEXT

The idea found in the code is any double dollar ($$) is being treated as
explicit dollar sign ($) instead of the start of a text object
reference. Due to missing update of the 's' variable, when creating a
text object for the text following the second dollar sign, the later was
prepended, leading to a double dollar in the output.

fix temporary printing of "(null)" for mpd vars

To minimise the chance of printing variables in an undefined state, call
free() as late as possible (i.e., right before strdup()'ing the new
value).

Couple minor fixes.

Make X11_initialise() work as it should, clarified IBM/Lenovo aka smapi
support in configure script.

Make print_config() stuff match configure.

Regenerate docs.

Adds the docs for the gauges

Signed-off-by: Brenden Matthews <brenden@rty.ca>

Fix: make sure compiling without X11 works

Fix possible mem leak with net stats.

Update changelog.

Merge branch 'master' of git.omp.am:/home/omp/git/conky

No limits for environment variables

Use dynamic allocated memory for skey and svalue,
allocate the length of value so that there are no limits,
but don't use the %a gnu extension to maintain compatibility

Fix wacky net speed values on startup.

Make gauge stuff look a little less funny.

Unbreak alias stuff.

It was broken by commit 98c64d8b3ea93a5a3ee098783b1399982f8c97a2, I did
a lowsy job of checking the patch over.

Update docs for $if_running.

Update configure.ac.in for 1.7.x.

Updating changelog.

I added Gagues to monitor variables

Signed-off-by: Brenden Matthews <brenden@rty.ca>

Disk i/o support on FreeBSD patch.

Patch sf.net id #2657227 (thanks Nikos).

Updating changelog.

Add extra_newline option for awesome's wiboxes

Hi Brenden,
Below is an updated patch, it works for me with the master.

p.s: don't forget to use git-am to apply the patch ;p

Signed-off-by: Brenden Matthews <brenden@rty.ca>

Fix compilation errors.

I hope I didn't break the alias stuff.

fix use_spacer in default config (sf.net id #2638653)

fix problems with negative numbers (sf.net id #2644593) (thanks Zhoushen Huang)

bugfix: segfault when nothing x11-related is mentioned before TEXT

Added alias configfile setting

This patch is written by a anonymous author, see patch on sf.net
id #2663691 . I changed it a bit so that it can't overwrite
existing environment variables and updated the docs and syntaxfiles

update README and manpage

fix padding of $processes

Padding to 4 digits allows for max 9999 processes, which should really
be more than enough for systems running conky. ;)

fix padding in temp_print

introduce percent_print() for printing percentages

rewrite human_readable() to minimise padding

Also fix spaced_print(): when given a width of e.g. 3, output at least 3
characters, not 2 as it was before.

if_updatenr added

move the custom defined memrchr to the right place

And make it static, as tailhead.c is the only user of it.

fix 966a05a9848f20a740c5e695d182039ebfe7d2fd

add help text for --print-config option

While here, fix also -C for builds with --disable-x11.

whitespace cleanup

remove x11 dependency of fonts

Instead, we just ignore them. Less errors in output, little less
ifdef's.

split off specials from conky.c

As a side effect, font and colour specific stuff got outsourced, too.
This is because the apropriate functions are used by conky.c as well as
specials.c, so they should be kept on their own.

In the long term I hope for positive impact on the X11 integration mess.
(Take e.g. the mass of X11 ifdefs cluttering conky.c.) Though this
commit contains no optimisations in this direction, just plain
outsourcing, to ensure minimised (intended: none) changes to conky's
interface behaviour.

add doxygen support

Since we have no doxygen compatible code comments (we hardly have
comments at all ;) there is not much documentation to be extracted. But
this is still useful, as doxy generates us nice callgraphs showing the
internal code flow chaos. :)

Usage is as follows: after having run ./autogen.sh, doxygen can be
triggered by simply issuing 'make doxy'. This will generate a doxy
config if none exists, and preset some stuff inside (especially
parsing of undocumented stuff). Before all consecutive invocations, the
doxy config (named 'Doxyfile') can be customised to one's personal
needs.

fix for building with --disable-x11

merge tail and head into a single source file

This allows having the same init-function for both objects. Code size
could be further reduced, as the print-functions share some code, too.

put tail and head code into separate files

In fact these two objects share a lot of code, so the bigger plan is to
merge them into a single file to share equal code.

This should not change anything to the code flow, besides making conky
startup a bit more robust due to less use of CRIT_ERR.

merge ibm-acpi and smapi together

Well, not really. I don't dare putting it all together into a single
source file, as that would just not make sense. Instead, this patch
eliminates the --enable-smapi configure option and all related checks,
replacing them with the equivalents from --enable-ibm.

outsource ibm-acpi stuff into it's own source file

Despite this is actually not much code and could stay inside linux.c as
well, this makes sense as I want to combine smapi and ibm-acpi
functionality. Virtually every user of a notebook made by IBM/Lenovo
will want both, so this should be a feature not a bug.

Besides making the ibm-acpi objects being built optionally, this should
not change anything to the code flow.

rewrite linux diskio code

Instead of using a hardcoded maximum number of slots for
stats of different disks, use a linked list. Also since the algorithm to
update each device's counters is the same for updating the totals, share
equal code, which in my eyes not only saves a bunch of LoC, but also
drastically increases readability.

fix potential segfault

The segfault can be triggered by using any diskio object with a
non-existent device, as prepare_diskio_stat() then returns 0 and the
call to obj->data.diskio->current in conky.c:4050 pulls the trigger.

In fact, it's not a problem when the device doesn't exist, as
update_diskio() simply won't fill in any values. So skip the check and
upon device node appearance everything goes it's normal way.

While there, also eliminate double readout of the last line of
/proc/diskstats: after the last line has been read, FEOF is not yet set.
BUT fgets() will return NULL when trying to read the next line. So
better check for fgets()'s return value instead of using feof().

Also strncmp() is useless here, since we really want to compare the full
paths. Besides, text_buffer_size also should be big enough to not make a
difference here.

fix coding style of the last few patches

Also, note that free() does not zero the passed pointer. So conditional
freeing always needs to look like this:
| if (var) {
|     free(var);
|     var = 0;
| }

No space between number and units please.

Null mpd variables should have an empty string patch sf.net id #2564747.

XMMS2 fix and refactoring patch sf.net id #2579357 (thanks Tamim).

Extended support for local Maildir patch sf.net id #2561323 (thanks Nicolas).

Fix shown battery status patch.

Fix shown battery status when the battery is not charging any more but is not full charged patch sf.net id #2556056 (thanks Martin).

Patch to clarify things in conky.conf sf.net id #2548805 (thanks Carpathia).

Fix nvidia memory frequency reading patch sf.net id #2493134.

Fix diskio_read/write patch sf.net id #2493084 (thanks Alexander).

Patch by Kim Holviala sf.net id #2484548.

* change the height of execbar and execibar to be the same as other
bars have by default (6 pixels)
* treat mixer values as percentages as they're usually 0-100 (switch
to spaced_print() with pad_percents)
* change temp_print() from snprintf() to spaced_print()
* remove decimals from temp_print() as none of the current sources can
supply values smaller than 1 degree (C or F, doesn't matter)
* add a space between number and the unit in human_readable()
* fix number printing in human_readable()
* network $upspeed and $downspeed now use human_readable()

Patch to add $battery_short variable.

Misc compilation fixes.

out_to_x can now be turned off

docs and syntax for overwrite_file and append_file

append_file output method

overwrite_file output method

Added output_to_stderr

Fix my last 'fix'.

Fix warning on older versions of gcc.

Couple fixes for the top_time stuff.

better combine top, top_mem and top_time handlers

Also free process list when ${top_time} is used

top.c: Allow heavier optimization

This makes it possible for the compiler to do better optimizations by
adding "static" to functions which do not need to be exported.

Since some of them (e.g. the compare_*() functions) are called very
often, this may decrease conky's need of resources a bit.

Don't do unneeded sp_acopy() calls

Update README and conky.1

With my last patch I did only update the XML documentation file without
updating the auto-generated manpage and README file.

This patch also updates README and conky.1.

Add ${top_time} sorting processes by CPU time

Build fix patch

This patch fixes build errors on some platforms due to missing includes.

simplify declaration of ifblock objects

Like OBJ_THREAD, OBJ_IF automatically calls obj_be_ifblock_if() when the
object matches.

use a global struct moc_s

Since there is no choice of which moc player to get information from,
all moc objects' data source can be identical. Also hide some internal
data (the thread e.g.). Since from now on there can only be one moc
thread (not highlander ;), we don't need to treat the thread object
specially.

While here, fix indenting.

separate parsing and interpreting text objects for certain objects

All objects parsing text into objects formerly parsed and evaluated
their arguments each update interval. This does only make sense when the
parsed text could change between updates, which is the case for execp
and execpi objects, but none of the others. So have them parse the text
when creating them, so each update interval only the output has to be
re-extracted.

In fact, this should give a performance boost when using any of the
affected objects, as parsing the input text is quite expensive since
we're evaluating templates.

move and add function prototypes to src/conky.c

do not use a global ifblock stack

Using a global ifblock stack for all parsed ifblock objects causes
problems when doing sub-parsing in objects taking other objects as
parameters, because the possibly non-empty stack at startup leads to
false alarm when checking for stack emptiness after parsing the objects.

Use a void ** as the object to pass around, so callers don't need to
know struct ifblock_stack_obj.

treat mpd opjects as non-threaded

In fact, they are. But we don't want to treat them as those, because
the rewrite made the only situation when threaded objects matter
uncritical, i.e. when an object calls calling parse_conky_vars().

add if_match object comparing strings, floats and ints

In general, argument types should match, but for combined long and
double usage the long is being converted to double before evaluation.

A few examples:
${if_match ${cpu} < 30}
${if_match "asdf" != "qwer"}
${if_match 0.5 < 0.50001}
${if_match 49.999 < 50}

fix segfault for diskiograph

The crux is to split args between those for scan_graph() on one hand and
prepare_diskio_stat() on the other hand. To make the code working with
minimal changes, move the yet optional devicename to the end.

General note on graphs:
- for all graphs there exist the optional arguments for scan_graph()
  which come first and are optional (intended duplicate)
- all other args are object specific, and may be optional as well

internal rewrite of mpd support

Handle mpd internal information inside mpd.c. Use a refcounter to check
if the mpd-information can be freed (maybe useless). Remove the now
useless "full" flag of free_text_objects.

Merge branch 'master' of git://nwl.cc/~n0-1/conky

Make gateway objects also detect routes with missing RTF_GATEWAY flag

This was a bug, not a feature. From the FreeBSD manpage rtentry(9):

| Vt struct sockaddr *rt_gateway;
|
| The "target" of the route, which can either represent a destination in
| its own right (some protocols will put a link-layer address here), or
| some intermediate stop on the way to that destination (if the
| RTF_GATEWAY flag is set).

So for a default route with "dest" and "mask" both zero, either
RTF_GATEWAY is present, or "gate" is zero.

Possible fix for minor IMAP idle bug.

Update docs.

Patch to allow escaping newlines in the config.

Merge branch 'master' of git://nwl.cc/~n0-1/conky

simplify human_readable

The only change of behaviour occurs when adjusting the unit of positive
values. For some reason 1000LL was used, which has now been replaced by
1024LL. Usage of abs() and MAX() might be OS dependent, but it should
not be too hard to implement them by hand if they're missing somewhere.

make use_spacer an enum

It was a static int before, holding the (already defined) enum's value,
which doesn't make sense. Also it's used only inside conky.c. Since
enums can't be assigned other values than what are defined, we can skip
checking for illegal spacer value at each invocation of spaced_print,
and then also drop the function name argument, which apparently didn't
make sense at all, because use_spacer is a global option.

simplify diskio stats

Instead of splitting information, use diskio_stats[0] in diskio.c for
the totals. This saves a few branches, and frees some data from struct
information.

Merge branch 'master' of git://nwl.cc/~n0-1/conky