* Use uniform print_pid_Xid to fix code duplication
* fix codesmell: no default in switch
* fix missing break after default
* fix missing in print_pid_Xid and replace if's by switch
* common.cc,diskio.cc: Display units in "SI" as per https://github.com/brndnmtthws/conky/issues/86 request
* Make the mem* and swap* variables to accept additional argument to print the usage in SI units
* common.cc: Fix indentation
* core.cc: Make the strndup() function call as manifest constant
* Add two more formal parameters to "human_readable()"
* diskio.cc: Restore the multiply of 1024LL
* Move the base multiplying into standalone function
* Use the "apply_base_multiplier()" function where necessary
* Try to amend #31
* BUILD_WLAN should be available for all OS.
Keep BUILD_WLAN OFF by default for compatibility reasons.
* WLAN-related variables should be available for every OS.
There are some problems (probably null-dereference)
* Fix $wireless_essid crashing conky if no argument provided.
Conky wasn't parsing the argument of the variable as it should, thus wasn't allocating the `dev` member variable.
Also fix some documentation stuff.
* Improve `get_freq` #20
Using the Intel® Power Gadget API (https://software.intel.com/en-us/blogs/2012/12/13/using-the-intel-power-gadget-api-on-mac-os-x) we can now get actual Core frequency and not the constant factory one.
Though, for some weird reason the API gives the same freq for all Cores, thus the |cpu| arg becomes useless.
* Oops, this accidently slipped in
* Introduce BUILD_IPGFREQ build option
This build option has been introduced for one particular reason:
On macOS getting current core-frequency is not supported by the APIs. A solution is to install Intel's ® Power Gadget which comes with an .app, a Framework and a kernel-extension. Though, this may trouble some alot, thus introduce BUILD_IPGFREQ.
* Forgot static here.
* Some improvements for get_freq again.
Fix frequency not printing correctly (I wasn't using the divisor)
Add more guards.
* Setup cmake files and project code for Objective-C code #17
We want to use CoreWLAN framework.
* update_cpu_usage() now supports multiple cores
Also, some cleanup.
* Updated default conky config to monitor Mac Networking
* Made Mac Friendly BuildOptions and generic default conky configs
* Undid Xdamage config and cleaned up previous IF statements
* Re-Added XDamage fix
* Finish up the algorithm. I think its now correct. Closes: #33
* Cleanup macro and introduce a no-op free_cpu() function for ALL cpu-related variables
free_cpu() must be implemented for every OS and on all except macOS its a no-op function.
* Reformat, add empty comment.
* Require at least glib-2.36 (from 03/2013) to remove deprecated g_type_init()
* Fix empty continuation line in Dockerfile
* Require at least libmicrohttpd 0.9.25 (from 09/2012) to replace deprecated MHD_create_response_from_data()
* fix versioncheck libmicrohttpd
* Missing char for null-terminator in print_v6addrs()
* safe_asprintf to fix /proc/subdir bufferoverflows
There is no max length of /proc/subdir, increasing the value of /proc/sys/kernel/pid_max is one way this can happen
* Fix conversion constant c++ string to variable char array
* Fix strncpy using unrelated MPD_BUFFER_MAX_LENGTH
* Use strncpy instead of strcpy in eve.cc
* Use strncpy instead of strcpy in libmpdclient.cc
* move safe_asprintf back and use ostreams in proc.cc to make sonarcloud happy
* Documentation fixes for macOS
This commit fixes documentation of a macOS specific variable (sip_status) as promised in #480.
Also, fix some identation problems in darwin.cc
* Amend #18 by polling less often! This will help lower CPU usage, too!
* Refactor mysql.cc
* Comparison of array temp not equal to a null pointer
* Switch and compare to C NULL instead std::nullptr_t to avoid compile error
* Use parentheses around addrs
* Remove parentheses around addrs
* Rebase
* Initial patches for Darwin support
* cmake files ready for building on Sierra. Dont know about other OSX versions
* Added darwin.h which will contain darwin specific headers
* Patches for enablin asprintf support (needs _POSIX_C_SOURCE >= __DARWIN_C_FULL so remove -D_POSIX_C_SOURCE=200809L). What is more, -D_LARGEFILE64_SOURCE is used by default so not needed on macOS
* Stubs for get_entropy_avail and get_entropy_poolsize
* More progress to getting to build
* More patches
* Now builds, remains fixing linking errors. Also implemented some functions
* Now links
* Now conky runs without sigabrt()'ing... An exception in semaphore.hh caused it to crash immediately upon initialisation
* README changes
* This was based on an old attempt I did with porting conky where I hadn't cloned the repo and thus it showed 3166 commits behind
* Patched a bit the semaphore.hh and now conky runs at 0.7% CPU usage and not in 100% or 200% as it did before! Yeah
* This update fixes swap-related miscalculations, though:
On macOS there can be used MANY swapfiles ( NOT swap partitions,,, the installation disk is the partition for storing swap memory data )
Thus conky code must be updated for Darwin to support multi-swapfile usage stats.
For now, we default to showing stats for swapfile0
* Fixed a bit the comments so they make sense
* Realised that the sysctl code can give us stats for the whole swap storage so removed the swapmode( int swapfd, unsigned long * retvail, unsigned long * retfree )
* I think the code is in good condition now
* semaphore.hh: Implemented ~semaphore() and added throwing exception
when sem initialisation fails just like in sem() for linux…
darwin.cc: partially implemented update_cpu_usage() and
update_total_processes()
* Implements update_total_processes()
( Now it finds the correct number of tasks )
the changes in cmake file don’t change the functionality… It is only
comments.
In upcoming updates lua support may be enabled and the comments are
related to lua support.
* get_cpu_count() now conforms to newer sysctl code as suggested by Apple ( we use hw.logicalcpumax instead of hw.ncpus ) --> The new implementation of get_cpu_count() finds the the max value of logical cores the mac could use on a boot.
update_total_processes() was patched to allow initialising only ONCE when conky runs the machHost and processorSet ( this could probably be implemented in a better way I guess though... )
* patch
* experimental support for get_top_info() and update_cpu_usage() :)
* going to patch stuff
* adds getting rss memory in get_top_info()
* fixes
* Couple things made more beautiful but too many things to fix ...
* Remove unneeded code from darwin_soundcard.h and keep only needed to
compile ( This way we are compatible to GPLv3, i think )
Include GPLv3 header in darwin.cc, darwin.h and darwin_soundcard.h
Add more beautiful and explanatory comments to ease the work of other
contributors… ( Mostly on things that don’t work! )
* Merged the macOS update_cpu_usage() implementation with the linux implementation but still it doesn't work as expected, plus, there are more memory related problems that will be fixed in newer release!
NOTE: It will be really good to see how functions are implemented on linux.cc for more complete implementation
* Now using a more linux-influenced implementation and dropping the old... ( only keeps mach stuff )
Needs some more work though to print accurate percentage! :)
* Add conky installer (.pkg) project
* Now gets thread count aswell! For now, we get this from update_total_processes... Also, removed the installer project because I need to learn more the software
* Properly get thread count!
- Patched core.cc to enable getting threads count on macOS!
- Cleanup in darwin.cc
* sort out some things in the code:
- running_threads_count is not the same as thread_count on macOS! There is actually a difference!
Correct a bit the code!
* Enable the if_mounted command and start implementing the check_mount function in darwin.cc
* Tighten things up
* This patch makes conky build easily and without patching on any supported macOS version! The new flag introduced is a temporary solution for allowing conky to compile on other OS without tweaking the cmake files. If you want to compile it with i18l support you will have to run the command: brew link gettext after you install gettext
* Update README to provide more complete info on building on macOS
* Fix
* patch check_mounted + SIP status
This commit:
- patches check_mounted function to match the implementation found in FreeBSD.cc! Now conky expects typing something like this:
conky -t '${if_mounted /Volumes/MacOS} ${endif}'
Please note that the correct way is "/Volumes/MacOS", not "/Volumes/MacOS/" ! You shouldnt put the backslash at the end... It will not work!
- Adds a new command called sip_status which you can use to make conky show System Integrity Protection related information such as if filesystem protection is enabled, if user approved kexts protection is enabled and more...
you can type conky -t '$sip_status' to see what it does...
( I will most likely update some things later regarding sip_status and check_mounted )
* Further implement sip_status
sip_status now has different functionality!
If you give no arguments it shows whether SIP is enabled or disabled ( generally )
If you give argument ( takes only ONE ) you can specify which specific thing of SIP you want to show status for...
You need to pass 0, 1, ..., 9 to $sip_status as argument to print status for apple-internal, untrusted-kexts, and more...
Below you can see the list:
ARG: RESULT:
nothing --> print enabled / disabled
0 --> allow_apple_internal
1 --> allow_untrusted_kexts
2 --> allow_task_for_pid
3 --> allow_unrestricted_fs
4 --> allow_kernel_debugger
5 --> allow_unrestricted_dtrace
6 --> allow_unrestricted_nvram
7 --> allow_device_configuration
8 --> allow_any_recovery_os
9 --> allow_user_approved_kexts
* sip_status progress & if_up enabled
This commit adds the 'a' argument for $sip_status which checks if the SIP configuration is ok or it is likely to break in the future.
Also, enabled the $if_up command, but there are some problems with it...
* sip_status patches
- Updated code based on newest version of csrstat utility (v.1.8)
The 1.8 version fixes a problem in finding status of ALLOW_DEVICE_CONFIGURATION flag.
- Fixed a bug where we mistakenly printed 'disabled' even when SIP was enabled.
* Disable ncurses checks in a way that will allow compiling for other OSes! ( it was a really quick patch before )
* More changes in cmake file to make code prettier and more exact
* More cmake files cuteness
* work for supporting versions prior Sierra
This patch brings more compatibility with versions prior to Sierra.
The project as is nearly links on a 10.9.5 OSX installation.
- on versions prior Sierra clock_gettime() is not implemented thus we need to provide our own implementation for conky to work. ( Currently we have minimal support, more studying needs to be done )
NOTE: In order to compile on versions as old as Mavericks you will need newer compilers because command line tools for 10.9.5 dont support some C++11 features... For this I donwloaded the commandline tools for Xcode 8.1 ( or 8 dont remember well ) and installed only the CommandLine Tools section with Pacifist!
( The OSX installer sees the OS as too old, so you need pacifist! )
* Oops
* $if_mounted ready
This patch removes the unneded print_mounted() function!
Also cleanup code.
* $if_up works
I think $if_up command works therefore remove the APPLE specific code in core.cc ( there wasnt anything different anyway ) and use the FreeBSD code.
Minor conky output changes in darwin.cc
* fix compatibility with Mavericks and before...
This update allows conky to be compiled on Mavericks and before by checking at compilation time what our build target is.
If it is Mavericks or lower, then cant use the csr_get_active_config func because it is not available.
On higher versions than Mavericks it works as before...
NOTE: For compiling on Mavericks or lower you will have to write 10.9 in CMAKE_OSX_DEPLOYMENT_TARGET when you run ccmake
* Added link for youtube video showing conky running on macOS
* Improvements
Minor improvements related to style
Removed unneeded free() calls
* Simplify Xcode configuration process! :)
* Removed update_cpu_usage() implementation
It was quite buggy so I removed it...
* Cleanup the source code
1. Bogus TODO comments have been deleted!
2. Started using DEBUG_MODE based printf
3. Explain some things better/cleanup
* Last small cleanup
The TODO in get_from_load_info() was unneeded. Removed!
Also, DEBUG_MODE is enabled for now...
* update authors file
* Improve README
Make it more clean and easier to understand.
* Update README.md
small tweak
* Work for conforming with "Development" rules
1. Add to variables.xml documentation about the $sip_status variable.
2. Add sip_status variable to conkyrc.vim and nanorc to allow highlighting
3. the friends in darwin.cc is ambiguous,,, some are more than friends.
* First patch set that implements get_top_info()
:)
* Calculate cpu usage of all processes more efficiently. (Now results match Activity Monitor's results a lot!!!)
More info about the get_top_info() implementation will come once I finish it! (soon)
* Cleanup patch!
o Move a couple things to better places
o Rename get_sample() to get_cpu_sample() to allow to be used by different functions such as get_cpu_usage() ( this is a scene from the future... )
o Improve the way we get cpu usage for each process by passing the already created proc_taskinfo struct to the appropriate function instead of creating another one
inside it!
* Correctly enable $running_processes variable
In our implementation update_top() doesn't calculate the count of running processes! We assign this functionality to the update_running_processes() function.
* Removed unneeded flag
This is not needed because we do not use update_top() for getting the number of running processes!
* Update README.md
* Update README.md
This adds support for TravisCL.
Though this will compile the project only on Linux it is a good sign to know if our port still can compile on Linux or we have broken things. It seems like it can't but I suspect the problem existed before my patches. I will investigate.
* First patch towards implementing #15
This patch a reworked get_top_info(). We now use the function get_top_info_for_kinfo_proc() to set the contents of each |process| struct.
Also, we calculate the total cpu usage and total process usage concurrently using GCD.
Aside from these we bring the function helper_get_proc_list() which makes the code simpler and cleaner.
* First fix for the algorithm that calculates % CPU usage of a process
The previous patch made the algorithm buggy. It required more reentrant-safe functions.
Make calc_cpu_total_r() which is a more reentrant-safe reimplementation of the calc_cpu_total() function.
This is not the last commit, more are coming in the future.
* Previous commit wasn't pushed...
* Problems committing
* Make the code cleaner but break the algorithm again #15
The algorithm doesn't work... It is a work in progress!
But this time we have merged the cpu_info and cpusample into one struct called cpusample
( I copied some variables from the linux implementation but they will probably be removed )
* Rework the code a bit / CPU usage % algorithm still doesn't work
Now get_from_load_info() is renamed to helper_update_threads_processes() and is called by both update_threads() and update_total_processes()
This makes the code nicer and removes the enum with the flags
* cleanup
remove unneeded functions, remove unneeded variables
* bugfix
* bugfix
* remove the whole get_top_info() code!
I will rework a lot this code so we can get rid of it.
* Work for #15
Now we use the ultimate solution for calculating % usage of CPU of a process!
Check the #15 Issue for information about this patch!
* Quick memory leak fix & some patches for the get_top_info algorithm
* quick patch for semaphore class
* Update the comments
* For consistency
* temp
* Revert "Merge branch 'master' into update_cpu_usage"
This reverts commit 7267a515c40d54b85107e51016703e6c10f3183e, reversing
changes made to 2190ff0b51463c12c61b3357cf846b8e705ebeb4.
* Revert "Revert "Merge branch 'master' into update_cpu_usage""
This reverts commit 692bd27a3adb2fe7027d65932467a140e42cc7bc.
* Partially implement the overall % CPU usage
* bug fix: I meant procs and threads!
* make the code better part1
* cleanup
* Stabilise the values of get_cpu_sample(). Affects get_top_info() and update_cpu_usage() ( fixes #13 and #15 )
For some reason the code that used host_statistics() function didn't produced bad values quite often. Replaced the function with host_processor_info() and changed the algorithm
and now the values are correct!
This affects the get_top_info() and update_cpu_usage() functions which wouldn't work correctly until now.
* Finish the optimised version of get_top_info()
Fixes #15
* Quick fix for #16
Probably I will come up with a different location for get_cpu_count() because I want to conform to linux implementation.
Also, the memory leak I am mentioning in get_cpu_count() may not actually exist, as I suspected in the back of my head.
* Strictness
* Cleanup ConkyPlatformChecks.cmake
Cleanup the quick patch code for Lua.
* Add warning to README
If you are using old version of cmake, update it. It will probably have problems when trying to locate lua.
* More cleanup
* Fix identation
This simplifies the diff.
Also, note that by merging the new code for checking for NCURSES we acheive better compatibility and easy build process
* Remove mixer related patches and files
With the newest patches from upstream, cmake checks if the header soundcard.h exists on the system.
If it doesn't exist (in the case of macOS for example) all mixer-related variables are disabled.
* Changes for using as submodule for ConkyX
* Fixes
* Implement get_freq
This implements #11
Though it doesn't support getting the **current** cpu frequency.
I haven't found a way to do it on macOS but i guess it can be done using a kernel extension. I may come back to this again in the future
* First work for correcting mem stats #6
Incorporate the update_pages_stolen() function and all of its dependencies.
I will see if I can patch the code in order to avoid license problems
* Work for #6
Now we follow the way apple's top works and not Activity Monitor.
For this reason we find different values. But correct based on top.
* cleanup for #6
* bug fix 1
* cleanup
* hmm sorry
* More cleanup
- Move the libtop_tsamp_t object to darwin.cc.
- Remove unneeded member variables.
* A bit documentation
* Implement secondary memory stats #6
Implement memwithbuffers, memeasyfree and memfree (which isn't secondary)
* Quit update_meminfo if libtop_tsamp_update_vm_stats() failed
* Update README.md
Revert to original conky readme
Move all macOS related information to the Wiki: https://github.com/Conky-for-macOS/conky-for-macOS/wiki
* Alot junk slipped in too!
* Merge net_stats branch
For now, I have only tested up / down bytes count on en0, and it works.
This is for #14
* Corrections
- Fix License headers in darwin* files
- Remove ConkyX entry from .gitignore which slipped into from the "forConkyX" branch which was used for another project ... ConkyX.
- Cleanup top.h (I will request the changes in a pull request finally)
* Fix identation and cleanup
* identation-fixes part1
* Try to fix problem compiling on linux
Improve checks for headers in ConkyPlatformChecks.cmake to amend the problem;
I mistakenly thought that statfs64 wasn't available in sys/statfs.h in macOS thus I tried to add platform specific code. Though, I add the checks in the wrong place and also messed up linux-specific code by replacing checks_include sys/statfs with check_symbol_exists statfs64
* attemp to completely fix linux problem!
* Fix #190:
* Added function `is_on_battery()` to correct set update interval
* Added setting `detect_battery` to correct select one or more batteries to check
* Added `detect_battery` to doc/config_settings.xml
In libical3, deprecated icaltime_from_timet is removed
Use icaltime_from_timet_with_zone instead. This function
exists in libical 2.x so no need to change requirements
* cmake: Add Haiku as a supported platform
* move mixer.cc to platform-specific as Haiku doesn't use OSS
* Add haiku.h with some stubs
* Use haiku.h
* Add some Haiku stubs
* Some Haiku network workarounds
It does not define SUN_LEN, provide a default.
It doesn't name the union in struct ifreq, use ifr_addr directly.
It seems Linux also declares it so it should work without change.
We can always define it for other platforms lacking it (add a cmake test ?).
Haiku does not have IFF_RUNNING, IFF_LINK seems close in semantics, so use it.
* Haiku: Minimal implementation of update_meminfo & get_top_info
* Haiku: implement update_cpu_usage
Some time after the last eve-related PR libcurl began returning
`CURLE_RECV_ERROR` from `curl_easy_perform()`, even when the request returns
good data that is otherwise usable.
Now, if the `struct xmlData` has been populated by write_data, we use whatever
came back (almost invariably a complete XML response). We don't depend on
libcurl returning CURLE_OK.
* Cleared TODO for the multi-GPU support
Added an optional parameter to the commands nvidia and nvidiabar.
If the parameter is omited default value is 0 for GPU0
The parameter is the GPU index starting at 0
Enhanced the cache system for multi-GPU support.
* Optimization and added modelname
* Update of documentation
* Update 2 of documentation
* Provide support to query the active port on PA's default sink
This commit exposes 2 objects:
- pa_sink_active_port_name
(example value: "analog-output-headphones")
- pa_sink_active_port_description
(example value: "Headphones")
that give respectively the name and description strings
for the active port on the default sink provided by PulseAudio.
* Reflect 9ed28f2a in the documentation
- For an AMD FX(tm)-6350 Six-Core Processor the file '/sys/.../present'
reports "0,3-7". I assume that chip is really an 8-core die with two
cores disabled... Presumably you could also get "0,3-4,6", and other
combos too...
* remove path from libirc-include
* feature: irc: possibly show last n lines persistently instead of all lines said since last time
* irc: add overflow-check, for case when no newmsg's ->next is NULL
Per manpage of mount(1) in newer util-linux:
The programs mount and umount traditionally maintained a list of
currently mounted filesystems in the file /etc/mtab. This real mtab
file is still supported, but on current Linux systems it is better
to make it a symlink to /proc/mounts instead, because a regular mtab
file maintained in userspace cannot reliably work with namespaces,
containers and other advanced Linux features.
In most new Linux (e.g. Debian) /etc/mtab is already symlinked to
/proc/mounts.
See
e778642a9e
for the gory details.
The code was multiplying the index with the size of the element, and
then adding it to the typed pointer (resulting in a double
multiplication and an OOB access).
Replace the buggy code with a slightly safer c++ alternative.
* eve: fix API, empty queue, conky reload, formatting issues
Since the EVE module was contributed, CCP has changed how the API works and
what the correct API URL is. So the module had to be updated to respect the
changes made on the EVE side.
Additionally, undefined behavior was occurring for characters without active
skill training queue, as is common for folks with multiple characters on a
single account, and was also occurring when Conky was reloaded following a
config file change. Now the `$eve` variable should just contain an empty
string if the character isn't training a skill (to make it detectable by
`$is_empty`), and the module should do The Right Thing (tm) when the
config file is changed while Conky is running.
The time remaining text has been adjusted to be slightly less verbose.
The module should do the right thing for up to 64 distinct characters.
Presently, though, every additional character requires a blocking CURL
request, meaning that querying many characters is presently not ideal. A
rewrite of the module to use asynchronous requests is probably in order, but
will be left for a future PR.
* eve: fix timezone handling; better handling of bad XML
* On Linux `gettimeofday()` doesn't return valid timezone data; we use
`localtime()` and the `tm_gmtoff` instead. Training times are no longer
incorrect as a result.
* Sometimes the EVE API will return spurious non-XML tags as part of the
response. This previously caused the XML parser to return a null document.
We tell libxml to `XML_PARSE_RECOVER` to try to recover from these errors
where they occur on real data.
* Implemented cached versions of get_nvidia_value and get_nvidia_string_balue. Unmutable values now only are queried once.
* Removed the TODO topic related to using static variable as cache + fixed a typo in the sample nvidia config
* Was using mtf variables to store the perf values. Fixed it
* Removed an useless else statement
* llabs() is included in FreeBSD since FreeBSD 5.0.
Oldest supported FreeBSD version is 9.3 now.
* stdio.h needs to be explicitly included here for FreeBSD.
I can wrap it in #ifdefs if needed, but I don't think it can hurt other OSes.
* addr config variable works fine on FreeBSD.
* FreeBSD 5.x is really an ancient version and long unsupported, so I think this check for it can be dropped.
* Fix and simplify code to get battery state and charge.
* Add required include on FreeBSD.
* Add needed include on FreeBSD.
* Also populate basename to avoid printing (null) process names.
Repored by fellow FreeBSD user Szabolcs Grof.
* nvidia: Update the docs and add more arguments
**Updated the docs in variables.xml to reflect the current nvidia object.**
Basically just copied the inline comments into the XML document
**Added the last of the compatible arguments**
The following arguments were added to `get_nvidia_barval`:
* memfree
* ambienttemp
* gpufreq
* memfreq
* mtrfreq
* imagequality
This should leave only incompatible arguments unsupported (e.g. min/max
values).
Decided to base percentages off of zero rather than the specified minimum
values.
Signed-off-by: Matt Sturgeon <matt@sturgeon.me.uk>
* nvidia: Better error/debug printing
* Merged `scan_nvidia_args` and `set_nvidia_type` into new
`set_nvidia_query`
* Added a translation array for `nvidia`, `nvidiabar`, `nvidiagauge` and
`nvidiagraph` commands (helps with error printing)
* Added some NORM_ERR prints to `get_nvidia_barval`, `get_nvidia_value` and
`get_nvidia_string`
* Removed exess newlines in nvidia's core.cc error definitions
Signed-off-by: Matt Sturgeon <matt@sturgeon.me.uk>
* nvidia: Better sanity checks when running queries
Introduced a pair of functions that checks whether the target we are
querying exists and how many of the target there are.
If the target doesn't exist there probably isn't a nvidia card
installed, if the target count query fails then the propietary drivers
probably aren't installed.
Fixes #269
Signed-off-by: Matt Sturgeon <matt@sturgeon.me.uk>
* Add native support for pulseaudio.
This adds ability to get Pulseaudio current volume without calling an
external program. Also it provides a few other parameters like card's
current profile similarly to pavucontrol. Currently it only queries the
default sink (and associated card).
* Switch to pulse async API.
This us by far the most efficient as conky subscribe to a few pulseaudio
notifications. Whenever pulseaudio server changes (volume or default
sink or default card), conky's callback updates the results.
This adds support for reading log entries from the Systemd journal. A
new command, called simply 'journal', is added which takes a number of
lines as a parameter and optionally which journal (user or system) to
read from.
Obviously, libsystemd is required to build journal support. Tested on
Arch Linux (Systemd v229).
Signed-off-by: Toke Høiland-Jørgensen <toke@toke.dk>
When parsing the argument string for graph objects, check if there is a
double-quoted command in the beginning of the string and extract it
before parsing any options. This prevents items in a command from being
improperly recognized as graph options. It should also eliminate the
need to place commands with spaces into a separate script.
Also, fall back on default_graph_{height,width} instead of using
hard-coded values so that the default values set in a user's config will
work.
