Bug 1321010 - Remove two minor bits of deritrus from IPDL Python code. r=kanru draft
authorAndrew McCreight <continuation@gmail.com>
Mon, 28 Nov 2016 15:16:57 -0800
changeset 446632 e69a0f4dca17af9d3649cdbf4e09f6fe0f52ad94
parent 446527 3853c539a1b7c803f1075d2c3ecefbdd314af1d8
child 538823 ea596b164e8702823c1c3ac43a9ef73076b3a6af
push id37834
push userbmo:continuation@gmail.com
push dateThu, 01 Dec 2016 17:09:24 +0000
reviewerskanru
bugs1321010
milestone53.0a1
Bug 1321010 - Remove two minor bits of deritrus from IPDL Python code. r=kanru The bug referred to in the comment was fixed 6 years ago. The includes in cgen.py are unused.
ipc/ipdl/ipdl/cgen.py
ipc/ipdl/ipdl/type.py
--- a/ipc/ipdl/ipdl/cgen.py
+++ b/ipc/ipdl/ipdl/cgen.py
@@ -1,16 +1,15 @@
 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 import os, sys
 
 from ipdl.ast import Visitor
-from ipdl.ast import IN, OUT, INOUT, ASYNC, SYNC, INTR
 
 class CodePrinter:
     def __init__(self, outf=sys.stdout, indentCols=4):
         self.outf = outf
         self.col = 0
         self.indentCols = indentCols
 
     def write(self, str):
--- a/ipc/ipdl/ipdl/type.py
+++ b/ipc/ipdl/ipdl/type.py
@@ -1445,18 +1445,16 @@ class CheckTypes(TcheckVisitor):
         if not mgstype.isManagedBy(ptype):
             self.error(
                 loc,
                 "|manages| declaration in protocol `%s' does not match any |manager| declaration in protocol `%s'",
                 pname, mgsname)
 
 
     def visitManager(self, mgr):
-        # FIXME/bug 541126: check that the protocol graph is acyclic
-
         pdecl = mgr.of.decl
         ptype, pname = pdecl.type, pdecl.shortname
 
         mgrdecl = mgr.decl
         mgrtype, mgrname = mgrdecl.type, mgrdecl.shortname
 
         # we added this information; sanity check it
         assert ptype.isManagedBy(mgrtype)