Andrew Cooke | Contents | Latest | RSS | Twitter | Previous | Next

C[omp]ute

Welcome to my blog, which was once a mailing list of the same name and is still generated by mail. Please reply via the "comment" links.

Always interested in offers/projects/new ideas. Eclectic experience in fields like: numerical computing; Python web; Java enterprise; functional languages; GPGPU; SQL databases; etc. Based in Santiago, Chile; telecommute worldwide. CV; email.

Personal Projects

Lepl parser for Python.

Colorless Green.

Photography around Santiago.

SVG experiment.

Professional Portfolio

Calibration of seismometers.

Data access via web services.

Cache rewrite.

Extending OpenSSH.

C-ORM: docs, API.

Last 100 entries

Psychology; [Bike] Exercise And Fuel; Continental Race King 2.2; VOIP quote for Cat Soft LLC; Removing Lowers; Mnesiacs; [Maths, Link] Dividing By Zero; [Book, Review] Ray Monk - Ludwig Wittgenstein: The Duty Of Genius; [Link, Bike, Computing] Evolving Lacing Patterns; [Jam] Strawberry and Orange Jam; [Chile, Privacy] Biometric Check During Mail Delivery; [Link, Chile, Spanish] Article on the Chilean Drought; [Bike] Extended Gear Ratios, Shimano XT M8000 (24/36 Chainring); [Link, Politics, USA] The Future Of American Democracy; Mass Hysteria; [Review, Books, Links] Kazuo Ishiguro - Never Let Me Go; [Link, Books] David Mitchell's Favourite Japanese Fiction; [Link, Bike] Rear Suspension Geometry; [Link, Cycling, Art] Strava Artwork; [Link, Computing] Useful gcc flags; [Link] Voynich Manuscript Decoded; [Bike] Notes on Servicing Suspension Forks; [Links, Computing] Snap, Flatpack, Appimage; [Link, Computing] Oracle is leaving Java (to die); [Link, Politics] Cubans + Ultrasonics; [Book, Link] Laurent Binet; VirtualBox; [Book, Link] No One's Ways; [Link] The Biggest Problem For Cyclists Is Bad Driving; [Computing] Doxygen, Sphinx, Breathe; [Admin] Brokw Recent Permalinks; [Bike, Chile] Buying Bearings in Santiago; [Computing, Opensuse] Upgrading to 42.3; [Link, Physics] First Support for a Physics Theory of Life; [Link, Bike] Peruvian Frame Maker; [Link] Awesome Game Theory Tit-For-Tat Thing; [Food, Review] La Fabbrica - Good Italian Food In Santiago; [Link, Programming] MySQL UTF8 Broken; [Link, Books] Latin American Authors; [Link, Computing] Optimizatin Puzzle; [Link, Books, Politics] Orwell Prize; [Link] What the Hell Is Happening With Qatar?; [Link] Deep Learning + Virtual Tensor Machines; [Link] Scaled Composites: Largest Wingspan Ever; [Link] SCP Foundation; [Bike] Lessons From 2 Leading 2 Trailing; [Link] Veg Restaurants in Santiago; [Link] List of Contemporary Latin American Authors; [Bike] FTHR; [Link] Whoa - NSA Reduces Collection (of US Residents); [Link] Red Bull's Breitbart; [Link] Linux Threads; [Link] Punycode; [Link] Bull / Girl Statues on Wall Street; [Link] Beautiful Chair Video; Update: Lower Pressures; [Link] Neat Python Exceptions; [Link] Fix for Windows 10 to Avoid Ads; [Link] Attacks on ZRTP; [Link] UK Jazz Invasion; [Review] Cuba; [Link] Aricle on Gender Reversal of US Presidential Debate; {OpenSuse] Fix for Network Offline in Updater Applet; [Link] Parkinson's Related to Gut Flora; Farellones Bike Park; [Meta] Tags; Update: Second Ride; Schwalbe Thunder Burt 2.1 v Continental X-King 2.4; Mountain Biking in Santiago; Books on Ethics; Security Fail from Command Driven Interface; Everything Old is New Again; Interesting Take on Trump's Lies; Chutney v6; References on Entropy; Amusing "Alexa.." broadcast; The Shame of Chile's Education System; Playing mp4 gifs in Firefox on Opensuses Leap 42.2; Concurrency at Microsoft; Globalisation: Uk -> Chile; OpenSuse 42.2 and Synaptics Touch-Pads; Even; Cherry Jam; Lebanese Writer Amin Maalouf; C++ - it's the language of the future; Learning From Trump; Chinese Writer Hu Fayun; And; Apricot Jam; Also; Excellent Article on USA Politics; Oh Metafilter; Prejudice Against The Rurals; Also, Zizek; Trump; Why Trump Won; Doxygen + Latex on CentOS 6; SMASH - Solve 5 Biggest Problems in Physics; Good article on racism, brexit, and social divides; Grandaddy are back!; Consciousness From Max Entropy

© 2006-2017 Andrew Cooke (site) / post authors (content).

Coding Guidelines for C

From: andrew cooke <andrew@...>

Date: Mon, 20 Jun 2011 10:29:38 -0400

I've been back coding in C for a while and want to record some simple rules
that make programs simpler:

1 - Always return a status.  Non-zero is an error.  Always check status on
    return.  On error, goto exit for cleanup.

    So functions should look like:

    int myfunction(blah blah) {
      char *text = NULL;
      void *point = NULL;
      int status = 0;
      ...
      if ((status = otherfunction(args here))) goto exit;
      if ((status = yetanother(...))) goto exit;
      ...
    exit:
      if (text) free(text);
      if (pointer) free(pointer)
      return status;
    }

2 - Write allocators and destructors for complex data (structs).  These should
    always take a pointer to a pointer and set the pointer to NULL.
    Similarly, use calloc to allocate data so that pointers are NULL from the
    start).  This follows on from the above - using NULL to indicate that data
    are already freed:

    int alloc_foo(foo **my_foo) {
      int status;
      *my_foo = calloc(1, sizeof(foo));
      if (! my_foo) return MEMORY_ERROR;
      (*my_foo)->counts = calloc(1, 20 * sizeof(int));
      if (! (*my_foo)->counts) return MEMORY_ERROR;
      if ((status = alloc_inner(&(*my_foo)->inner))) return status
      return 0;
    }

    int free_foo(foo **my_foo, status) {
      if (*my_foo) {
        if ((*my_foo)->counts) free((*my_foo)->counts);
        (*my_foo)->counts = NULL;
	status = free_inner(&(*my_foo)->inner, status)
        *my_foo = NULL;
      }
      return status;
    }

3 - Cleanup functions should take status an argument and "make it worse".  the
    the idea here is that you want to call cleanup functions even if there are
    errors, but you don't want to "reset" status to OK (zero) if it was bad.
    In the examples above it's not so clear, but consider, say, releasing a
    database:

    int release_db(cnxn *db, int status) {
      db_lib_release(db);
      if (db_lib_error() && ! status) status = DB_ERROR;
      return status;
    }

    This simplifies the "exit:" code and also allows one-liners when you want
    to do one thing with a database and then release it:

      if ((status = release_db(db, do_db_thing(db, ....)))) goto exit;

4 - Use typedefs for structs:

    typedef struct {
      int my_int;
    } my_struct;

5 - Either namespace with names (and use static functions to avoid clutter
    wherever possible) or namespace with "objects":

    typedef struct self {
      ...
      int (*free)(struct self **foo, int status);
    } foo;

Andrew

Comment on this post