This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 433336 - Use boolean primitives in ModelAssembler.runProcessors() method
Summary: Use boolean primitives in ModelAssembler.runProcessors() method
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.4   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 4.4 M7   Edit
Assignee: Lars Vogel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-23 15:34 EDT by Lars Vogel CLA
Modified: 2014-04-29 09:29 EDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2014-04-23 15:34:16 EDT 

    


    


      



        
          Comment 1
        

        
          Lars Vogel CLA

        

        
        

        
          2014-04-23 15:35:38 EDT
        

      






https://git.eclipse.org/r/25434

    


    


      



        
          Comment 2
        

        
          Lars Vogel CLA

        

        
        

        
          2014-04-23 15:42:29 EDT
        

      






Fixed with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=822529afe037f0f0cedd1407584f5990e2a4e997

Thanks to Tom for the pointer

    


    


      



        
          Comment 3
        

        
          Lars Vogel CLA

        

        
        

        
          2014-04-23 17:53:03 EDT
        

      






(In reply to Lars Vogel from comment #2)
> Fixed with
> http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/
> ?id=822529afe037f0f0cedd1407584f5990e2a4e997
> 
> Thanks to Tom for the pointer

Merge was waiting for another change, rebased onto origin/master and merged with 

https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=eb34ba54fbade1382ae7de9f259e19d977acbb58

    


    


      



        
          Comment 4
        

        
          Dani Megert CLA

        

        
        

        
          2014-04-25 12:00:54 EDT
        

      






This fix was not good. See bug 433339 for details.

    


    


      



        
          Comment 5
        

        
          Lars Vogel CLA

        

        
        

        
          2014-04-29 09:29:17 EDT
        

      






After the fix of Bug 433339 all relevant unit tests pass. I mark this one as verified in Build id: I20140427-2030.