Avoid crash in WOLoginItem

It's quite possible that this code doesn't work any more (it is
using now-deprecated APIs), but this at least fixes the crash.

Fix Doxygen annotations

Fix warning about loss of precision in implicit cast

We were casting from NSUInteger (unsigned long) to unsigned.

Update README and future-proof copyright notices

Merge branch 'pull/1'

Originally submitted at: https://github.com/wincent/WOPublic/pull/1

* pull/1:
  Add basic readme
  Fix submodule URLs

Add basic readme

Changes applied to original PR (https://github.com/wincent/WOPublic/pull/1):

- Fix whitespace.

Signed-off-by: Greg Hurrell <greg@hurrell.net>

Fix submodule URLs

Fix crasher

Hopefully the final step in my painful journey from GC to ARC.

Signed-off-by: Greg Hurrell <greg@hurrell.net>

Fix use-after-free crasher

Stack trace here:

  https://wincent.com/comments/11596

At the same time, switched to best practice for assigning to instance
variables in `init` methods.

Signed-off-by: Greg Hurrell <greg@hurrell.net>

Use explicit self in WOLoginItemList init

This has always been best practice in `init` methods, but I have been
inconsistent in following it. This SO post explains the difference:

http://stackoverflow.com/a/19665933/1626737

Effectively, after `self = [super init]`, the explicit "self" referred
to in `self->listRef` may not be the same as the implicit "self" in a
naked `listRef` expression.

People have been reporting crash on launch:

- issue: https://wincent.com/issues/2173
- see also comments here: http://www.macupdate.com/app/mac/10042/synergy

I am not sure if this will fix that issue, but it might, so it seems
worth giving it a try. If it does fix it, I should audit all init
methods in the code base and make sure they do the right thing.

Signed-off-by: Greg Hurrell <greg@hurrell.net>

Fix leaks left over from ARC conversion

Signed-off-by: Greg Hurrell <greg@hurrell.net>

Apply changes for ARC-compatibility

Signed-off-by: Greg Hurrell <greg@hurrell.net>

Silence another Clang warning about incompatible types

This looks like it was a bug; there don't appear to be any reasonable
circumstances under which we'd want to return an NSMutableArray here
(there is no inheritance relationship or common interface).

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Silence some Clang warnings about format string markers

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update WO_EXPORT_CLASS macro for 64-bit Mountain Lion

The old `.no_dead_strip` doesn't work when targeting x86_64 on Mountain
Lion, but the good news is that a simpler trick that doesn't require
inline assembly now does work.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Ignore "build" directory created by Xcode

Signed-off-by: Greg Hurrell <greg@causes.com>

Git: ignore user-specific project bundle insertions added by Xcode 4.1

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Fix a (cosmetic) typo in a code documentation comment

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Move stringification macros into separate versioning header

The versioning header also includes macros for embedding RCS ID strings
into object files.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add "Submodules" group for submodule folder references

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Xcode project file buzz

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Improve grouping within main() of benchmark tool

Make the grouping of test variants a little more consistent.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add more benchmarks for NSArray (WORubyBlocks)

To clearly demonstrate that the overhead of block dispatch is not
significant, add benchmarks involving repeated calls to the "map:"
method. Sample results:

  1,000,000-element array, 1 iteration
       -[NSArray map:]: 3.444701/0.089042/3.533743 (user/system/total)
    manual enumeration: 3.545396/0.093752/3.639148 (user/system/total)
      fast enumeration: 3.437787/0.093748/3.531535 (user/system/total)
  1000-element array, 1000 iterations
       -[NSArray map:]: 2.913624/0.079141/2.992765 (user/system/total)
    manual enumeration: 2.987299/0.084068/3.071367 (user/system/total)
      fast enumeration: 2.911901/0.082780/2.994681 (user/system/total)

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add benchmarks tool

Add tool for running some simple benchmarks. The first benchmark added
is a comparison of the speed of using the new -[NSArray map:] method
with a couple of variants of explicit manual enumeration.

     -[NSArray map:]: 3.451480/0.092237/3.543717 (user/system/total)
  manual enumeration: 3.562207/0.093985/3.656192 (user/system/total)
    fast enumeration: 3.479888/0.124764/3.604652 (user/system/total)

The results should be taken with a grain of salt as multiple runs can
produce a both relative (changes of order) and absolute variation, but
at least at first glance it would appear that for non
performance-critical applications the election of method really isn't
that important.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Remove unused "Other Sources" group from Xcode project

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add license headers to WOUsageMeter files

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add references to new files to Xcode project

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Initial import of WOUsageMeter class from WOCommon

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add Ruby-esque "map:" method to NSArray

The method takes a block parameter, taking advantage of the new blocks
support added by the Mac OS X 10.6 tool chain and runtime.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Remove deprecated calls from unit tests

Signed-off-by: Wincent Colaiuta <win@wincent.com>

WOProcessLifetime: silence compiler warning

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Fix bug in error-path of pathForFolder:domain:create:

This bug was discovered while running the unit tests on a new system
with a clean Snow Leopard install. Previously the error-path containing
the bug had never been exercised.

Flesh out the unit tests to increase the likelihood of such a bug being
caught in the future if it ever returns (it is not possible to
guarantee 100% that the error-path can be triggered, because we don't
know what directories will exist on the system where the tests are
running).

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Trim documentation to further discourage use of deprecated method

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Deprecate the createDirectoryAtPath:attributes:createParents: method

This method is made redundant by the createDirectoryAtPath:
withIntermediateDirectories:attributes:error: method which as added to
NSFileManager in Leopard, so turn it into a simple wrapper for that
method and mark it as deprecated.

This fixes deprecation warnings building against the 10.6 SDK (not
tested against the 10.5 SDK) about the use of createDirectoryAtPath:
attributes: method which is currently deprecated.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add required categories information to NSString URL Utilities header

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update import statements and surrounding comments

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update license header in newly added files

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add references to new category files to Xcode project

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Import NSString URL utilities and NSURL convenience category

Both of these are coming across from the WOCommon repository.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Fix leak in -[WOLoginItem addToList:]

Missing call to WOMakeCollectable/CFMakeCollectable.

Discovred using static analysis.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update buildtools submodule 8fb8129..93e282e

  93e282e Switch to Snow Leopard SDK
  04230fd README: update for Snow Leopard
  1bf82fa Drop PowerPC support
  d49b922 README: add note on tags
  aa40fda README: remove unnecessary references to enclosing directory
  6044395 README: add notes on repository URLs and layout
  2ff2a43 README: remove "included tools" list
  64c03c6 README: indent paths to make them stand out
  18ae5c3 Update README for buildtools as a Git submodule
  40f3da0 Fix GPL references in the README
  9b7d87b Rewrap README to 80 columns
  366d169 Remove legacy $Id$ tag

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Link to SystemConfiguration framework

This is required by the newly added WOHost class.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Fix compilation errors in WOHost

Update import statements and replace "unless" macro (which doesn't exist
in WOPublic and won't be added either).

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add references to WOHost files to Xcode project

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Reformat WOHost code documentation to match WOPublic style

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add BSD license header to WOHost files

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Import WOHost files from WOCommon

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update URL to more recent version of documentation

Signed-off-by: Wincent Colaiuta <win@wincent.com>

WOMemoryBarrier.h: drop PowerPC macros on Snow Leopard branch

This branch is for projects which run only on Snow Leopard, and as Snow
Leopard won't even run on PowerPC processors, the ppc macros are just
cruft which clutter up the header so drop them.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add WOProcessManager references to Xcode project file

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Reformat WOProcessManager files for WOPublic

A number of clean-ups:

  - add BSD license header
  - change references from "WOCommon" to "WOPublic"
  - update import statements
  - rewrap long code documentation lines

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Initial import of WOProcessManager class (from WOCommon)

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update WOProcessLifetime headers

Add BSD license header, change WOCommon references to WOPublic, and
update import statements.

WOProcessManager is the remaining dependency that will have to come
across to WOPublic in order for this to be compilable.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add WOProcessLifetime references to Xcode project

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add WOEnumerate.h reference to Xcode project file

So it can be included in project-wide search index and the like.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Initial import of WOProcessLifetime class from WOCommon

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update wmalloc reference to emalloc

wmalloc was renamed to emalloc back in 32cff7d, so update the stale
reference in WOSysctl; with this change, the WOPublic test suite now
builds and passes again.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Formatting changes for WOSysctl and WOProcessSerialNumber

Minor cosmetic changes to bring the code into line with the rest of
WOPublic:

  - wrap or reformat long lines
  - remove blank lines at end of file

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update WOSysctl and WOProcessSerialNumber headers

Include BSD license header, updated copyright year ranges, and trim some
fat.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add Xcode project references to new files

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Initial import of WOSysctl and WOProcessSerialNumber classes

These are being migrated over from WOCommon.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update URL broken by issue tracker move

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Rewrap Doxygen comments in WOEnumerate.h

For consistency with the other headers in the repo, wrap code comments
to 80 columns.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update WOEnumerate.h header

Add BSD license text, update copyright range etc.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Import WOEnumerate.h from WOCommon

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Replace hardcode instances of "WOCommon" with "WOPublic"

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add missing files to project

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Fix header breakage

The build chain was broken by the changes in buildtools, so these header
errors weren't picked up at commit time. Now the buildchain is restored
again these errors were detected and are fixed by this commit.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update references to use submodule version of buildtools

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add buildtools as a submodule

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Raise if WOLoginItem initialized with a nil path

Initialization is one of the "edge" boundaries of an API where we want
to be strict about what we will accept in order to detect errors sooner.

I just ran into a crash caused by a build bug (missing file) which in
turn caused a nil value to be passed into the WOLoginItem initialization
codepath; this parameter check would have helped me discover the cause
of the problem a little sooner.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add WOLoginItem* classes to Xcode project file

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Fix copy/paste bug in WO_STRINGIFY_CONTENTS macro

This macro somehow got mangled during the migration into WOPublic.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Guard against redefinition of WO_STRINGIFY and WO_STRINGIFY_CONTENTS

There are many products which try to define the same macros, so guard
against re-definition warnings and errors.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update names in WOLoginItem

Some of the names that were previously used when this class was in
WOCommon need to be updated for WOPublic.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Import WO_FREE macro from WOCommon

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Clean up WOLoginItem and WOLoginItem for consistency with WOPublic

Update headers (update copyright ranges, remove unnecessary lines, add
license notice) and do some minor re-wrapping for better display on
narrow terminals.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Import WOLoginItem and WOLoginItemList from WOCommon

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Use emalloc in NSFileManager path utilities class

Update to sync with new name added in commit 32cff7d.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Rename wmalloc() to emalloc()

Follow the same pattern as shown in the "xcalloc" function: the mnemonic
here is "Exception malloc" (cf "eXit calloc").

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add xcalloc() wrapper

This is a wrapper for the calloc() function in the standard library
which, in the event of a failure, prints an error message to the
standard error ("error: failed to calloc X bytes") and calls exit().

Mnemonic: "eXit calloc".

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add license header to NSArray WOConvenience category

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Import NSArray WOConvenience category from WOCommon

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Reformat WOLogManager Doxygen comments to fit under 80 columns

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Use custom Doxygen footer

Rather than showing the Doxygen icon, show a discrete textual link
instead.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Doxygen: update DOCSET configuration

Use project-specific settings for these new variables (added
by Doxygen 1.5.9) rather than using the defaults provided by
Doxygen.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Use JavaScript in Doxygen-generated documentation

Makes the documentation a little more user-friendly.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Instruct Doxygen to inline sources

Doesn't seem to work, at least with Doxygen 1.5.9, but activate
it anyway.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add example to WOLog macro documentation

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Use Doxygen 1.5.9 to update Doxyfile

This fixes a bunch of warnings when using the old file with a newer
version of Doxygen:

  Warning: Tag `USE_WINDOWS_ENCODING' at line 34 of file Doxyfile has become obsolete.
  To avoid this warning please update your configuration file using "doxygen -u"
  Warning: Tag `DETAILS_AT_TOP' at line 56 of file Doxyfile has become obsolete.
  To avoid this warning please update your configuration file using "doxygen -u"
  Warning: Tag `MAX_DOT_GRAPH_WIDTH' at line 244 of file Doxyfile has become obsolete.
  To avoid this warning please update your configuration file using "doxygen -u"
  Warning: Tag `MAX_DOT_GRAPH_HEIGHT' at line 245 of file Doxyfile has become obsolete.
  To avoid this warning please update your configuration file using "doxygen -u"

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Documentation tweaks to silence Doxygen warnings and errors

Many errors went away in the move from Doxygen 1.5.7.1 to
1.5.9, but these changes were necessary to get a warning and
error-free build.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Update Doxyfile to point to new Graphviz installation

This is the new Graphviz 2.14 installation based on the installer
package at:

  http://www.ryandesign.com/graphviz/

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add Documentation (Doxygen) target

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Remove unused main.c file

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Add unit test target

This target builds the bundle and executes the tests using WOTest.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Customize Info.plist

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Trim fat from WOConvenienceMacros code documentation

The information about the enclosing braces is too obvious to warrant
inclusion.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Augment documentation for WO_DICTIONARY and WO_MUTABLE_DICTIONARY

Note that these macros depend on code that is provided by a category
on NSDictionary.

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Migrate WO_DICTIONARY and WO_MUTABLE_DICTIONARY macros from WOCommon

Signed-off-by: Wincent Colaiuta <win@wincent.com>

Whitespace fix in NSDictionary WOCreation category

Remove trailing whitespace.

Signed-off-by: Wincent Colaiuta <win@wincent.com>