| 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

Transcript of German Scientists on Learning of Hiroshima; Calvert Journal; Telephone System Quotes for Cat Soft LLC; Owen Jones on Twitter; Telephone System Quotes for Cat Soft LLC; Possible Japanese Authors; Complex American Literature; Chutney v5; Weird Componentized Virus; Interesting Argentinian Author - Antonio Di Benedetto; Useful Thread on MetaPhysics; RAND on fighting online anarchy (2001); Now Is Cat Soft LLC's Chance To Save Up To 32% On Mail; NSA Hacked; Call Center Services for Cat Soft LLC; Very Good LRB Article on Brexit; Nussbaum on Anger; Credit Card Processing for Cat Soft LLC; Discover new movies on demand in our online cinema; Tasting; Credit Card Processing for Cat Soft LLC; Apple + Kiwi Jam; Hit Me; Increase Efficiency with GPS Vehicle Tracking for Cat Soft LLC; Sudoku - CSP + Chaos; Recycling Electronics In Santiago; Vector Displays in OpenGL; Call Center Services for Cat Soft LLC; And Anti-Aliased; OpenGL - Render via Intermediate Texture; And Garmin Connect; Using Garmin Forerunner 230 With Linux; Payroll Service Quotes for Cat Soft LLC; (Beating Dead Horse) StackOverflow; Current State of Justice in China; Now Is Cat Soft LLC's Chance To Save Up To 32% On Mail; Axiom of Determinacy; Ewww; Fee Chaos Book; Course on Differential Geometry; Increase Efficiency with GPS Vehicle Tracking for Cat Soft LLC; Okay, but...; Sparse Matrices, Deep Learning; Sounds Bad; Applebaum Rape; Tomato Chutney v4; Have to add...; Culturally Liberal and Nothing More; Weird Finite / Infinite Result; Your diamond is a beaten up mess; Maths Books; Good Bike Route from Providencia / Las Condes to Panul\; Iain Pears (Author of Complex Plots); Plum Jam; Excellent; More Recently; For a moment I forgot StackOverflow sucked; A Few Weeks On...; Chilean Book Recommendations; How To Write Shared Libraries; Jenny Erpenbeck (Author); Dijkstra, Coins, Tables; Python libraries error on OpenSuse; Deserving Trump; And Smugness; McCloskey Economics Trilogy; cmocka - Mocks for C; Concept Creep (Americans); Futhark - OpenCL Language; Moved / Gone; Fan and USB issues; Burgers in Santiago; The Origin of Icosahedral Symmetry in Viruses; autoenum on PyPI; Jars Explains; Tomato Chutney v3; REST; US Elections and Gender: 24 Point Swing; PPPoE on OpenSuse Leap 42.1; SuperMicro X10SDV-TLN4F/F with Opensuse Leap 42.1; Big Data AI Could Be Very Bad Indeed....; Cornering; Postcapitalism (Paul Mason); Black Science Fiction; Git is not a CDN; Mining of Massive Data Sets; Rachel Kaadzi Ghansah; How great republics meet their end; Raspberry, Strawberry and Banana Jam; Interesting Dead Areas of Math; Later Taste; For Sale; Death By Bean; It's Good!; Tomato Chutney v2; Time ATAC MX 2 Pedals - First Impressions; Online Chilean Crafts; Intellectual Variety; Taste + Texture; Time Invariance and Gauge Symmetry; Jodorowsky

© 2006-2015 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