Incorrect output for repeated qualified fields

[ewencp]

If you import another pbj file and then try to use one of its messages as a repeated field, e.g.:

import "Loc.pbj";
message foo {
    optional Sirikata.Protocol.Loc.BulkLocationUpdate loc_update = 2;
}

it generates bad output like this:

        Sirikata::Protocol::Loc::LocationUpdate IPintoResponse::loc_update(int index) const {
            return Sirikata::Protocol::Loc::LocationUpdate(*const_cast<_Protocol_::Sirikata.Protocol.Loc.LocationUpdate*>(&super->loc_update(index)));
        }

where the names are not properly converted. There's a bunch of if blocks selecting based on "isMultiplicitiveAdvancedType" (which I am not sure what it even indicates), "isRepeated", and "isMessage"/"isSubmessage". I think they all generate correct code, but maybe we're not setting one of the flags correctly.