You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by je...@apache.org on 2012/12/11 23:40:30 UTC
git commit: THRIFT-1765 Incorrect error message printed for null or
negative keys
Updated Branches:
refs/heads/master 741e715e8 -> 7740739b7
THRIFT-1765 Incorrect error message printed for null or negative keys
Project: http://git-wip-us.apache.org/repos/asf/thrift/repo
Commit: http://git-wip-us.apache.org/repos/asf/thrift/commit/7740739b
Tree: http://git-wip-us.apache.org/repos/asf/thrift/tree/7740739b
Diff: http://git-wip-us.apache.org/repos/asf/thrift/diff/7740739b
Branch: refs/heads/master
Commit: 7740739b79d0e694d6a15fa000c30bae028d8785
Parents: 741e715
Author: Jens Geyer <je...@apache.org>
Authored: Tue Dec 11 23:38:12 2012 +0100
Committer: Jens Geyer <je...@apache.org>
Committed: Tue Dec 11 23:38:12 2012 +0100
----------------------------------------------------------------------
compiler/cpp/src/thrifty.yy | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/thrift/blob/7740739b/compiler/cpp/src/thrifty.yy
----------------------------------------------------------------------
diff --git a/compiler/cpp/src/thrifty.yy b/compiler/cpp/src/thrifty.yy
index 696fd46..0a7f184 100644
--- a/compiler/cpp/src/thrifty.yy
+++ b/compiler/cpp/src/thrifty.yy
@@ -960,7 +960,7 @@ FieldIdentifier:
* warn if the user-specified negative value isn't what
* thrift would have auto-assigned.
*/
- pwarning(1, "Negative field key (%d) differs from what would be "
+ pwarning(1, "Nonpositive field key (%"PRIi64") differs from what would be "
"auto-assigned by thrift (%d).\n", $1, y_field_val);
}
/*
@@ -971,7 +971,7 @@ FieldIdentifier:
$$.value = $1;
$$.auto_assigned = false;
} else {
- pwarning(1, "Nonpositive value (%d) not allowed as a field key.\n",
+ pwarning(1, "Nonpositive value (%"PRIi64") not allowed as a field key.\n",
$1);
$$.value = y_field_val--;
$$.auto_assigned = true;