The best way to do something like this is to create a pull request. This will not only give you credit for the change and make the code easy to add to Phobos, but it will make sure it doesn’t break any existing tests.
Please have a look here: http://wiki.dlang.org/Pull_Requests <http://wiki.dlang.org/Pull_Requests> -Steve > On Aug 25, 2015, at 7:15 PM, Q. Schroll via phobos <[email protected]> > wrote: > > I'm new to this, maybe this proposal has been made once and I've overseen it. > > Tuples only allow the %s format specifier which is rather useless except from > debugging. > > One way to format output a Tuple is expanding it. But this works only, when > the tuple is outermost, i. e. like in format("%d: %.3f", > int_float_tuple.expand), and not like in format("%(%d: %.3f%)", > int_float_tuple_array.map!(t => t.expand)) which currently doesn't work. > > Therefore I propose to make Tuples also to be formated by %( %) and %( %| %). > For the first one, the expression inside is used to format the tuple the in > the aforementioned way. > The second takes the tuple like an array (one expression and separators, not > the escaping stuff). > > unittest > { > import std.format : format; > import std.typecons : Tuple, tuple; > alias t = tuple; > auto if_list = [ t(1, 1.0), t(2, 4.0), t(3, 9.0) ]; > > assert(format("%s", t("a", 1)) == `Tuple!(string, > int)("a", 1)`); > assert(format("%(%#x v %.4f w %#x%)", t(1, 1.0, 10)) == `0x1 v 1.0000 w > 0xa`); > assert(format("%(q%sq%| x %)", t("abc", 1, 2.3, [4,5])) == `qabcq x q1q x > q2.3q x q[4, 5]q`); > assert(format("%(%(%d^2 = %.1f%); %)", if_list) == `1^2 = 1.0; > 2^2 = 4.0; 3^2 = 9.0`); > } > > Both can be done. I already have tested it and it works perfectly. > > All you have to do: > In typecons.d, line 762, add std.format.FormatSpec!char fmt = "%s" > parameter. > In the function body, add at front > if (fmt.nested) > { > if (fmt.sep) > { > foreach (i, Type; Types) > { > static if (i > 0) > { > sink(fmt.sep); > } > static if (is(Type == class) && is(typeof(Type.init) == > shared)) > { > sink(Type.stringof); > } > else > { > import std.format : format; > sink(format(fmt.nested, field[i])); > } > } > } > else > { > import std.format : format; > sink(format(fmt.nested, expand())); > } > return; > } > The rest of the function can be left as it is. > > This feature cannot break any code I can imagine. > _______________________________________________ > phobos mailing list > [email protected] > http://lists.puremagic.com/mailman/listinfo/phobos
_______________________________________________ phobos mailing list [email protected] http://lists.puremagic.com/mailman/listinfo/phobos
