763238 - "Opens" and "Bridge" declarations need to use fully-qualified names

Status: RESOLVED FIXED

(Core :: IPC, defect)

Description

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

We might be bridging/opening protocols in other C++ namespaces, so fq names need to be generated.

Comment 1

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Attachment: Use fully-qualified names for Open/Bridge

Comment 2

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Attachment: Test for bug 763228

Comment 3

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Comment on attachment 631701 [details] [diff] [review]
Use fully-qualified names for Open/Bridge

Review of attachment 631701 [details] [diff] [review]:
-----------------------------------------------------------------

::: ipc/ipdl/ipdl/lower.py
@@ +447,1 @@
>          return Type(t.name())

Hm, you repeat this pattern in lots of places. How about you just have another method that does this:

  def _internalName(self, thing):
    return thing.fullname() if self.fq else thing.name()

Then all the other places would be much simpler, don't you think? Up to you I guess.

@@ +481,5 @@
>      def visitMessageType(self, m): assert 0
>      def visitVoidType(self, v): assert 0
>      def visitStateType(self, st): assert 0
>  
> +def _cxxBareType(ipdltype, side, fq=0):

False? 0 and 1 are kinda 1980's style ;)

Comment 4

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

(In reply to ben turner [:bent] from comment #3)
> Comment on attachment 631701 [details] [diff] [review]
> Use fully-qualified names for Open/Bridge
> 
> Review of attachment 631701 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: ipc/ipdl/ipdl/lower.py
> @@ +447,1 @@
> >          return Type(t.name())
> 
> Hm, you repeat this pattern in lots of places. How about you just have
> another method that does this:
> 
>   def _internalName(self, thing):
>     return thing.fullname() if self.fq else thing.name()
> 
> Then all the other places would be much simpler, don't you think? Up to you
> I guess.
> 

Good idea.  Added, with typename() helper.

> @@ +481,5 @@
> >      def visitMessageType(self, m): assert 0
> >      def visitVoidType(self, v): assert 0
> >      def visitStateType(self, st): assert 0
> >  
> > +def _cxxBareType(ipdltype, side, fq=0):
> 
> False? 0 and 1 are kinda 1980's style ;)

Well, that's the prevailing style in this file.  I guess 1980s programmers got much more work done from not having to type as much ;).  I didn't change this because it's the prevailing style.  Feel free to file a followup for changing that, and assign to bent ;).

Comment 5

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

https://hg.mozilla.org/integration/mozilla-inbound/rev/cdc00d148906
https://hg.mozilla.org/integration/mozilla-inbound/rev/a9479d148ce5

Comment 6

[Graeme McCutcheon [:graememcc]]

https://hg.mozilla.org/mozilla-central/rev/37e50965e1b5
https://hg.mozilla.org/mozilla-central/rev/cdc00d148906

Comment 7

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Not sure why we're retriaging closed bugs, but this would have blocked (blocks a blocker).