{"thread":{"id":"21197","subject":"[PATCH 2/3] git config: clarify bool types","startedAt":"2009-10-11T20:46:10Z","lastAt":"2009-10-13T10:02:15Z","messageCount":15,"participants":["Felipe Contreras","Thiago Farina","Junio C Hamano","Michael J Gruber"],"isPatch":true,"patchVersion":1,"patchTotal":3},"messages":[{"id":"124638","messageId":"1255293973-17444-1-git-send-email-felipe.contreras@gmail.com","threadId":"21197","inReplyTo":null,"subject":"[PATCH 0/3] Trivial patches","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-11T20:46:10Z","receivedAt":"2009-10-11T20:46:10Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"These are a bunch of no-brainers. The one for bool types has been discussed and\nagreed before.\n\nFelipe Contreras (3):\n  Fix a bunch of pointer declarations (codestyle)\n  git config: clarify bool types\n  user-manual: use 'fast-forward'\n\n Documentation/user-manual.txt |   14 +++++++-------\n builtin-config.c              |    4 ++--\n diff.c                        |    2 +-\n 3 files changed, 10 insertions(+), 10 deletions(-)\n"},{"id":"124640","messageId":"1255293973-17444-2-git-send-email-felipe.contreras@gmail.com","threadId":"21197","inReplyTo":"1255293973-17444-1-git-send-email-felipe.contreras@gmail.com","subject":"[PATCH 1/3] Fix a bunch of pointer declarations (codestyle)","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-11T20:46:11Z","receivedAt":"2009-10-11T20:46:11Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Essentially; s/type* /type */ as per the coding guidelines.\n\nSigned-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n---\n diff.c |    2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/diff.c b/diff.c\nindex e1be189..b39c1b6 100644\n--- a/diff.c\n+++ b/diff.c\n@@ -999,7 +999,7 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)\n \t       total_files, adds, dels);\n }\n \n-static void show_shortstats(struct diffstat_t* data, struct diff_options *options)\n+static void show_shortstats(struct diffstat_t *data, struct diff_options *options)\n {\n \tint i, adds = 0, dels = 0, total_files = data->nr;\n \n-- \n1.6.5.4.g31fc3\n"},{"id":"124636","messageId":"1255293973-17444-3-git-send-email-felipe.contreras@gmail.com","threadId":"21197","inReplyTo":"1255293973-17444-2-git-send-email-felipe.contreras@gmail.com","subject":"[PATCH 2/3] git config: clarify bool types","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-11T20:46:12Z","receivedAt":"2009-10-11T20:46:12Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"The value is what it is, the --bool and --bool-or-int options don't\nspecify the value type, just how it is interpreted. For example: a value\nof '1' can be interpreted as 'true'.\n\nSigned-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n---\n builtin-config.c |    4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/builtin-config.c b/builtin-config.c\nindex a2d656e..29d7b75 100644\n--- a/builtin-config.c\n+++ b/builtin-config.c\n@@ -66,9 +66,9 @@ static struct option builtin_config_options[] = {\n \tOPT_STRING(0, \"get-color\", &get_color_slot, \"slot\", \"find the color configured: [default]\"),\n \tOPT_STRING(0, \"get-colorbool\", &get_colorbool_slot, \"slot\", \"find the color setting: [stdout-is-tty]\"),\n \tOPT_GROUP(\"Type\"),\n-\tOPT_BIT(0, \"bool\", &types, \"value is \\\"true\\\" or \\\"false\\\"\", TYPE_BOOL),\n+\tOPT_BIT(0, \"bool\", &types, \"value is intepreted as bool (\\\"true\\\" or \\\"false\\\")\", TYPE_BOOL),\n \tOPT_BIT(0, \"int\", &types, \"value is decimal number\", TYPE_INT),\n-\tOPT_BIT(0, \"bool-or-int\", &types, \"value is --bool or --int\", TYPE_BOOL_OR_INT),\n+\tOPT_BIT(0, \"bool-or-int\", &types, \"value is interpreted either as bool or int\", TYPE_BOOL_OR_INT),\n \tOPT_GROUP(\"Other\"),\n \tOPT_BOOLEAN('z', \"null\", &end_null, \"terminate values with NUL byte\"),\n \tOPT_END(),\n-- \n1.6.5.4.g31fc3\n"},{"id":"124637","messageId":"1255293973-17444-4-git-send-email-felipe.contreras@gmail.com","threadId":"21197","inReplyTo":"1255293973-17444-3-git-send-email-felipe.contreras@gmail.com","subject":"[PATCH 3/3] user-manual: use 'fast-forward'","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-11T20:46:13Z","receivedAt":"2009-10-11T20:46:13Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"It's a compound word.\n\nSigned-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n---\n Documentation/user-manual.txt |   14 +++++++-------\n 1 files changed, 7 insertions(+), 7 deletions(-)\n\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex 67ebffa..42d92fd 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -1384,7 +1384,7 @@ were merged.\n \n However, if the current branch is a descendant of the other--so every\n commit present in the one is already contained in the other--then git\n-just performs a \"fast forward\"; the head of the current branch is moved\n+just performs a \"fast-forward\"; the head of the current branch is moved\n forward to point at the head of the merged-in branch, without any new\n commits being created.\n \n@@ -1719,7 +1719,7 @@ producing a default commit message documenting the branch and\n repository that you pulled from.\n \n (But note that no such commit will be created in the case of a\n-<<fast-forwards,fast forward>>; instead, your branch will just be\n+<<fast-forwards,fast-forward>>; instead, your branch will just be\n updated to point to the latest commit from the upstream branch.)\n \n The `git pull` command can also be given \".\" as the \"remote\" repository,\n@@ -1943,7 +1943,7 @@ $ git push ssh://yourserver.com/~you/proj.git master\n -------------------------------------------------\n \n As with `git fetch`, `git push` will complain if this does not result in a\n-<<fast-forwards,fast forward>>; see the following section for details on\n+<<fast-forwards,fast-forward>>; see the following section for details on\n handling this case.\n \n Note that the target of a \"push\" is normally a\n@@ -1976,7 +1976,7 @@ details.\n What to do when a push fails\n ~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n \n-If a push would not result in a <<fast-forwards,fast forward>> of the\n+If a push would not result in a <<fast-forwards,fast-forward>> of the\n remote branch, then it will fail with an error like:\n \n -------------------------------------------------\n@@ -2115,7 +2115,7 @@ $ git checkout release && git pull\n \n Important note!  If you have any local changes in these branches, then\n this merge will create a commit object in the history (with no local\n-changes git will simply do a \"Fast forward\" merge).  Many people dislike\n+changes git will simply do a fast-forward merge).  Many people dislike\n the \"noise\" that this creates in the Linux history, so you should avoid\n doing this capriciously in the \"release\" branch, as these noisy commits\n will become part of the permanent history when you ask Linus to pull\n@@ -2729,9 +2729,9 @@ In the previous example, when updating an existing branch, \"git fetch\"\n checks to make sure that the most recent commit on the remote\n branch is a descendant of the most recent commit on your copy of the\n branch before updating your copy of the branch to point at the new\n-commit.  Git calls this process a <<fast-forwards,fast forward>>.\n+commit.  Git calls this process a <<fast-forwards,fast-forward>>.\n \n-A fast forward looks something like this:\n+A fast-forward looks something like this:\n \n ................................................\n  o--o--o--o <-- old head of the branch\n-- \n1.6.5.4.g31fc3\n"},{"id":"124642","messageId":"a4c8a6d00910111353r20de1fb6u2289999491a9310a@mail.gmail.com","threadId":"21197","inReplyTo":"1255293973-17444-2-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH 1/3] Fix a bunch of pointer declarations (codestyle)","fromName":"Thiago Farina","fromEmail":"tfransosi@gmail.com","sentAt":"2009-10-11T20:53:42Z","receivedAt":"2009-10-11T20:53:42Z","isPatch":true,"sender":{"key":"tfransosi@gmail.com","avatar":"https://avatars.githubusercontent.com/u/970071?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> wrote:\n  \"Fix a bunch of pointer declarations\".\n\nBut in this patch you fixed *only* one pointer declaration:\nstruct diffstat_t* data -> struct diffstat_t *data\n"},{"id":"124643","messageId":"94a0d4530910111356x35dbb340o7779f63e7a0d0ff7@mail.gmail.com","threadId":"21197","inReplyTo":"a4c8a6d00910111353r20de1fb6u2289999491a9310a@mail.gmail.com","subject":"Re: [PATCH 1/3] Fix a bunch of pointer declarations (codestyle)","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-11T20:56:51Z","receivedAt":"2009-10-11T20:56:51Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Sun, Oct 11, 2009 at 11:53 PM, Thiago Farina <tfransosi@gmail.com> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> wrote:\n>  \"Fix a bunch of pointer declarations\".\n>\n> But in this patch you fixed *only* one pointer declaration:\n> struct diffstat_t* data -> struct diffstat_t *data\n\nYeah, it's a remainder of a patch I sent some time ago. I don't know\nwhy that didn't got picked up =/\n\n-- \nFelipe Contreras\n"},{"id":"124660","messageId":"7v7hv1kxyg.fsf@alter.siamese.dyndns.org","threadId":"21197","inReplyTo":"1255293973-17444-3-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH 2/3] git config: clarify bool types","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-12T05:01:11Z","receivedAt":"2009-10-12T05:01:11Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> The value is what it is, the --bool and --bool-or-int options don't\n> specify the value type, just how it is interpreted. For example: a value\n> of '1' can be interpreted as 'true'.\n\nIt is not really about \"interpreting\", but about showing, isn't it?\n\nWith this in your .git/config file:\n\n    [core]\n        repositoryformatversion = 0\n\nyou would see\n\n    $ git config --bool-or-int core.repositoryformatversion\n    0\n    $ git config --bool core.repositoryformatversion\n    false\n    $ git config --int core.repositoryformatversion\n    0\n\nSo it would be more like...\n\n\tshow value as boolean\n        show value as integer or boolean\n        show value as integer\n\nwouldn't it?\n\n> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n> ---\n>  builtin-config.c |    4 ++--\n>  1 files changed, 2 insertions(+), 2 deletions(-)\n>\n> diff --git a/builtin-config.c b/builtin-config.c\n> index a2d656e..29d7b75 100644\n> --- a/builtin-config.c\n> +++ b/builtin-config.c\n> @@ -66,9 +66,9 @@ static struct option builtin_config_options[] = {\n>  \tOPT_STRING(0, \"get-color\", &get_color_slot, \"slot\", \"find the color configured: [default]\"),\n>  \tOPT_STRING(0, \"get-colorbool\", &get_colorbool_slot, \"slot\", \"find the color setting: [stdout-is-tty]\"),\n>  \tOPT_GROUP(\"Type\"),\n> -\tOPT_BIT(0, \"bool\", &types, \"value is \\\"true\\\" or \\\"false\\\"\", TYPE_BOOL),\n> +\tOPT_BIT(0, \"bool\", &types, \"value is intepreted as bool (\\\"true\\\" or \\\"false\\\")\", TYPE_BOOL),\n>  \tOPT_BIT(0, \"int\", &types, \"value is decimal number\", TYPE_INT),\n> -\tOPT_BIT(0, \"bool-or-int\", &types, \"value is --bool or --int\", TYPE_BOOL_OR_INT),\n> +\tOPT_BIT(0, \"bool-or-int\", &types, \"value is interpreted either as bool or int\", TYPE_BOOL_OR_INT),\n>  \tOPT_GROUP(\"Other\"),\n>  \tOPT_BOOLEAN('z', \"null\", &end_null, \"terminate values with NUL byte\"),\n>  \tOPT_END(),\n> -- \n> 1.6.5.4.g31fc3\n"},{"id":"124659","messageId":"7veip9jj6z.fsf@alter.siamese.dyndns.org","threadId":"21197","inReplyTo":"1255293973-17444-4-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH 3/3] user-manual: use 'fast-forward'","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-12T05:05:24Z","receivedAt":"2009-10-12T05:05:24Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> It's a compound word.\n\nThanks.\n\nThis removes 7 out of 37 \"fast forward\" in Documentation/ directory.  Are\nwe going to convert the remaining ones, too?\n"},{"id":"124682","messageId":"94a0d4530910120303y205e6cfeg80ba0dfd6ed5a045@mail.gmail.com","threadId":"21197","inReplyTo":"7v7hv1kxyg.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 2/3] git config: clarify bool types","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-12T10:03:35Z","receivedAt":"2009-10-12T10:03:35Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Mon, Oct 12, 2009 at 8:01 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>> The value is what it is, the --bool and --bool-or-int options don't\n>> specify the value type, just how it is interpreted. For example: a value\n>> of '1' can be interpreted as 'true'.\n>\n> It is not really about \"interpreting\", but about showing, isn't it?\n\nUnless you are setting it, instead of reading it.\n\n-- \nFelipe Contreras\n"},{"id":"124683","messageId":"94a0d4530910120305o49ed7c74sf6a03097b1098a5f@mail.gmail.com","threadId":"21197","inReplyTo":"7veip9jj6z.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 3/3] user-manual: use 'fast-forward'","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-12T10:05:04Z","receivedAt":"2009-10-12T10:05:04Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Mon, Oct 12, 2009 at 8:05 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>> It's a compound word.\n>\n> Thanks.\n>\n> This removes 7 out of 37 \"fast forward\" in Documentation/ directory.  Are\n> we going to convert the remaining ones, too?\n\nSure. Once I finish pushing the patches I have for user-manual, which\nis the one I'm interested right now.\n\n-- \nFelipe Contreras\n"},{"id":"124691","messageId":"4AD3216B.7030507@drmicha.warpmail.net","threadId":"21197","inReplyTo":"94a0d4530910120303y205e6cfeg80ba0dfd6ed5a045@mail.gmail.com","subject":"Re: [PATCH 2/3] git config: clarify bool types","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2009-10-12T12:30:35Z","receivedAt":"2009-10-12T12:30:35Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Felipe Contreras venit, vidit, dixit 12.10.2009 12:03:\n> On Mon, Oct 12, 2009 at 8:01 AM, Junio C Hamano <gitster@pobox.com> wrote:\n>> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>>\n>>> The value is what it is, the --bool and --bool-or-int options don't\n>>> specify the value type, just how it is interpreted. For example: a value\n>>> of '1' can be interpreted as 'true'.\n>>\n>> It is not really about \"interpreting\", but about showing, isn't it?\n> \n> Unless you are setting it, instead of reading it.\n> \n\nI'd still suggest fixing the typo (\"interpreted\") and spelling out\n\"boolean\".\n\nMichael\n"},{"id":"124693","messageId":"4AD325FF.6070105@drmicha.warpmail.net","threadId":"21197","inReplyTo":"94a0d4530910120305o49ed7c74sf6a03097b1098a5f@mail.gmail.com","subject":"Re: [PATCH 3/3] user-manual: use 'fast-forward'","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2009-10-12T12:50:07Z","receivedAt":"2009-10-12T12:50:07Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Felipe Contreras venit, vidit, dixit 12.10.2009 12:05:\n> On Mon, Oct 12, 2009 at 8:05 AM, Junio C Hamano <gitster@pobox.com> wrote:\n>> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>>\n>>> It's a compound word.\n>>\n>> Thanks.\n>>\n>> This removes 7 out of 37 \"fast forward\" in Documentation/ directory.  Are\n>> we going to convert the remaining ones, too?\n> \n> Sure. Once I finish pushing the patches I have for user-manual, which\n> is the one I'm interested right now.\n> \n\nAnd how about the other 40 or so in the remaining code?\n\nIn fact, hyphenated and separated forms of this compound noun are almost\nevenly split, which reflects well upon the fact that there is no\nclear-cut rule in US English (and neither in BE).\n\nSo, if you are suggesting a style we should define it and follow it\nconsistently. Our main source of definitions is the glossary. Now, that\nhas a truly salomonic solution:\n\nfast forward\n\tA fast-forward is a special type...\n\nI suggest that this is the first place which needs attention, after\nfixating the definition, maybe not just among the three of us, all\nnon-native speakers.\n\nMichael\n"},{"id":"124719","messageId":"94a0d4530910121014k666207b9ub38fcecd47641ace@mail.gmail.com","threadId":"21197","inReplyTo":"4AD3216B.7030507@drmicha.warpmail.net","subject":"Re: [PATCH 2/3] git config: clarify bool types","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-12T17:14:49Z","receivedAt":"2009-10-12T17:14:49Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Mon, Oct 12, 2009 at 3:30 PM, Michael J Gruber\n<git@drmicha.warpmail.net> wrote:\n> Felipe Contreras venit, vidit, dixit 12.10.2009 12:03:\n>> On Mon, Oct 12, 2009 at 8:01 AM, Junio C Hamano <gitster@pobox.com> wrote:\n>>> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>>>\n>>>> The value is what it is, the --bool and --bool-or-int options don't\n>>>> specify the value type, just how it is interpreted. For example: a value\n>>>> of '1' can be interpreted as 'true'.\n>>>\n>>> It is not really about \"interpreting\", but about showing, isn't it?\n>>\n>> Unless you are setting it, instead of reading it.\n>>\n>\n> I'd still suggest fixing the typo (\"interpreted\") and spelling out\n> \"boolean\".\n\nOops! You mean s/intepreted/interpreted/?\n\nIf we spell 'boolean' we might as well spell 'integer'; I think bool\nand int are fine.\n\n-- \nFelipe Contreras\n"},{"id":"124818","messageId":"4AD427AD.3010701@drmicha.warpmail.net","threadId":"21197","inReplyTo":"94a0d4530910121014k666207b9ub38fcecd47641ace@mail.gmail.com","subject":"Re: [PATCH 2/3] git config: clarify bool types","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2009-10-13T07:09:33Z","receivedAt":"2009-10-13T07:09:33Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Felipe Contreras venit, vidit, dixit 12.10.2009 19:14:\n> On Mon, Oct 12, 2009 at 3:30 PM, Michael J Gruber\n> <git@drmicha.warpmail.net> wrote:\n>> Felipe Contreras venit, vidit, dixit 12.10.2009 12:03:\n>>> On Mon, Oct 12, 2009 at 8:01 AM, Junio C Hamano <gitster@pobox.com> wrote:\n>>>> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>>>>\n>>>>> The value is what it is, the --bool and --bool-or-int options don't\n>>>>> specify the value type, just how it is interpreted. For example: a value\n>>>>> of '1' can be interpreted as 'true'.\n>>>>\n>>>> It is not really about \"interpreting\", but about showing, isn't it?\n>>>\n>>> Unless you are setting it, instead of reading it.\n>>>\n>>\n>> I'd still suggest fixing the typo (\"interpreted\") and spelling out\n>> \"boolean\".\n> \n> Oops! You mean s/intepreted/interpreted/?\n\nYep :)\n\n> \n> If we spell 'boolean' we might as well spell 'integer'; I think bool\n> and int are fine.\n> \n\n\"int\" is at least a standard type name in C, whereas \"bool\" is not; but,\nyes, feel free to spell out \"integer\", or use \"--int or --bool\" as it\nis, which is a back reference to the corresponding entries for \"--int\"\nand \"--bool\", where things should be spelled out.\n\nMichael\n"},{"id":"124834","messageId":"94a0d4530910130302j26c5308ew5266d03c775b6ab0@mail.gmail.com","threadId":"21197","inReplyTo":"4AD427AD.3010701@drmicha.warpmail.net","subject":"Re: [PATCH 2/3] git config: clarify bool types","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2009-10-13T10:02:15Z","receivedAt":"2009-10-13T10:02:15Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Tue, Oct 13, 2009 at 10:09 AM, Michael J Gruber\n<git@drmicha.warpmail.net> wrote:\n> Felipe Contreras venit, vidit, dixit 12.10.2009 19:14:\n>> On Mon, Oct 12, 2009 at 3:30 PM, Michael J Gruber\n>> <git@drmicha.warpmail.net> wrote:\n>>> Felipe Contreras venit, vidit, dixit 12.10.2009 12:03:\n>>>> On Mon, Oct 12, 2009 at 8:01 AM, Junio C Hamano <gitster@pobox.com> wrote:\n>>>>> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>>>>>\n>>>>>> The value is what it is, the --bool and --bool-or-int options don't\n>>>>>> specify the value type, just how it is interpreted. For example: a value\n>>>>>> of '1' can be interpreted as 'true'.\n>>>>>\n>>>>> It is not really about \"interpreting\", but about showing, isn't it?\n>>>>\n>>>> Unless you are setting it, instead of reading it.\n>>>>\n>>>\n>>> I'd still suggest fixing the typo (\"interpreted\") and spelling out\n>>> \"boolean\".\n>>\n>> Oops! You mean s/intepreted/interpreted/?\n>\n> Yep :)\n>\n>>\n>> If we spell 'boolean' we might as well spell 'integer'; I think bool\n>> and int are fine.\n>>\n>\n> \"int\" is at least a standard type name in C, whereas \"bool\" is not; but,\n> yes, feel free to spell out \"integer\", or use \"--int or --bool\" as it\n> is, which is a back reference to the corresponding entries for \"--int\"\n> and \"--bool\", where things should be spelled out.\n\n'bool' is also a standard type, in C99.\n\n-- \nFelipe Contreras\n"}]}