May 04, 2008 Prettiful error messages with templates | ||||
|---|---|---|---|---|
  | ||||
I'm currently using this somewhat verbose approach to template error messages:
debug
{
  int opAdd(U, int line = __LINE__, string file = __FILE__)(U args)
  {
    static if (__traits(compiles, somethingHere)) // check if this template should have been instantiated at all
    {
      pragma(msg, "Failed instantiating template in file " ~ file ~ " on line " ~ toString(line)); // for a more prettiful error message. Could easily put this in the assert
      static assert(0);
    }
    // Do actual work here. Don't use line or file arguments.
  }
}
else
{
  int opAdd(U)(U args)
  {
    // Do actual work here.
  }
}
Now, what are the problems with this approach?
1. It is rather verbose.
2. Code duplication.
3. Template bloat for the debug version. For every time + is used, a new instantiation is created. I have duplicated the code to avoid this in release builds.
4. toString(int) is not CTFE-compatible.
Good things:
It gives exact error messages, telling the user what went wrong and where.
So, what do I want?
An easier way of doing this. If it weren't for #3 (which might be a limitation of the linker or compiler), I could avoid the code duplication, so I guess that's the biggest problem.
Any ideas?
-- Simen
 | ||||
May 05, 2008 Re: Prettiful error messages with templates | ||||
|---|---|---|---|---|
  | ||||
Posted in reply to Simen Kjaeraas  | Not sure how this works in D2 and it doesn't solve the biggest problems but here's a possible begin:
char[] assertCompiles(uint line, char[] file, char[] code)
{
    return `debug { static if (!__traits(compiles, ` ~ code ~ `))`
            `{`
                `pragma(msg, "Failed instantiating template in file ` ~ file
~ ` on line ` ~ ctfeToString(line) ~ `");`
                `static assert(false);`
            `}}`;
}
iirc I adapted ctfeToString from phobos' std.string:
const char[10] digits    = "0123456789";            /// 0..9
/// ditto
char[] ctfeToString(ulong u)
{
    char[] buffer;
    auto buflen = ulong.sizeof * 3;
    while (buflen--)
        buffer ~= ' ';
    int ndigits;
    ndigits = 0;
    if (u < 10)
    // Avoid storage allocation for simple stuff
        return digits[u .. u + 1];
    else
    {
        while (u)
        {
            uint c = (u % 10) + '0';
            u /= 10;
            ndigits++;
            buffer[buffer.length - ndigits] = cast(char)c;
        }
        return buffer[buffer.length - ndigits .. buffer.length].dup;
    }
}
it'll help a bit with the verbosity, used as:
int opAdd(U, uint line = __LINE__, char[] file = __FILE__ ) (U args)
{
    mixin( assertCompiles(line, file, code));
    ...
}
Will you post back when you found some good solutions? Thanks.
Good luck and happy hacking ;)
 | |||
Copyright © 1999-2021 by the D Language Foundation
 
Permalink
Reply