diff options
| author | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-26 23:27:18 +0000 |
|---|---|---|
| committer | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-26 23:27:18 +0000 |
| commit | ba054c05e3ff36022f013549d91c60e8df06ef0d (patch) | |
| tree | fd48d8792ef5dbe7989d72daff8f236d4334661d /ppapi/generators/idl_thunk.py | |
| parent | 8fb9e2339d695d9e0a0e021d1dec06ee0368ea76 (diff) | |
| download | chromium_src-ba054c05e3ff36022f013549d91c60e8df06ef0d.zip chromium_src-ba054c05e3ff36022f013549d91c60e8df06ef0d.tar.gz chromium_src-ba054c05e3ff36022f013549d91c60e8df06ef0d.tar.bz2 | |
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
Diffstat (limited to 'ppapi/generators/idl_thunk.py')
| -rwxr-xr-x | ppapi/generators/idl_thunk.py | 6 |
1 files changed, 2 insertions, 4 deletions
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 |