From ba054c05e3ff36022f013549d91c60e8df06ef0d Mon Sep 17 00:00:00 2001 From: "teravest@chromium.org" Date: Tue, 26 Feb 2013 23:27:18 +0000 Subject: IDL: Fix assertion error in idl_thunk. Recently, some nodes in the IDL AST started reporting multiple builds in their build_list. I don't understand what caused this behavior change, but this patch treats those cases more gracefully. BUG= Review URL: https://chromiumcodereview.appspot.com/12328108 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@184767 0039d316-1c4b-4281-b951-d872f2087c98 --- ppapi/generators/idl_thunk.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'ppapi/generators/idl_thunk.py') diff --git a/ppapi/generators/idl_thunk.py b/ppapi/generators/idl_thunk.py index 1f99658..b8adb2e 100755 --- a/ppapi/generators/idl_thunk.py +++ b/ppapi/generators/idl_thunk.py @@ -302,15 +302,13 @@ def _IsNewestMember(member, members, releases): releases - The set of releases to check for versions in. """ build_list = member.GetUniqueReleases(releases) - assert(len(build_list) == 1) - release = build_list[-1] # Pick the newest release. + release = build_list[0] # Pick the oldest release. same_name_siblings = filter( lambda n: str(n) == str(member) and n != member, members) for s in same_name_siblings: sibling_build_list = s.GetUniqueReleases(releases) - assert(len(sibling_build_list) == 1) - sibling_release = sibling_build_list[-1] + sibling_release = sibling_build_list[0] if sibling_release > release: return False return True -- cgit v1.1