Ignore:
Timestamp:
2014-10-15T00:24:51+02:00 (10 years ago)
Author:
donvip
Message:

[josm_plugins] fix some Sonar issues

File:
1 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/videomapping/src/org/openstreetmap/josm/plugins/videomapping/video/VideoEngine.java

    r29769 r30722  
    66import java.util.LinkedList;
    77import java.util.List;
     8
     9import org.openstreetmap.josm.Main;
    810
    911import uk.co.caprica.vlcj.binding.internal.libvlc_media_t;
    … …  
    4850                                WindowsRuntimeUtil.VLC_INSTALL_DIR_KEY);
    4951                    } catch (RuntimeException e) {
    50                         System.err.println(e.getMessage());
     52                        Main.error(e);
    5153                    }
    5254                }
    … …  
    5860           
    5961        if (vlcInstallDir != null) {
    60             System.out.println("videomapping: found VLC install dir: "+vlcInstallDir);
     62            Main.info("videomapping: found VLC install dir: "+vlcInstallDir);
    6163            NativeLibrary.addSearchPath("libvlc", vlcInstallDir);
    6264        } else {
    63             System.err.println("videomapping: unable to locate VLC install dir");
     65            Main.error("videomapping: unable to locate VLC install dir");
    6466        }
    6567        }
    6668       
    67         public VideoEngine(Window parent)
    68         {
     69        public VideoEngine(Window parent) {
    6970                System.setProperty("logj4.configuration","file:log4j.xml"); //TODO still unsure if we can't link this to the JOSM log4j instance
    70                 videos = new LinkedList<Video>();
    71                 observers = new LinkedList<VideosObserver>();
     71                videos = new LinkedList<>();
     72                observers = new LinkedList<>();
    7273                try {
    7374                        mediaPlayerFactory = new MediaPlayerFactory(libvlcArgs);
    7475                fullScreenStrategy = new DefaultFullScreenStrategy(parent);
    7576                } catch (NoClassDefFoundError e) {
    76             System.err.println(tr("Unable to find JNA Java library!"));
     77                    Main.error(tr("Unable to find JNA Java library!"));
    7778        } catch (UnsatisfiedLinkError e) {
    78             System.err.println(tr("Unable to find native libvlc library!"));
    79         } catch (Throwable t) {
    80             System.err.println(t.getMessage());
     79            Main.error(tr("Unable to find native libvlc library!"));
     80        } catch (Exception t) {
     81            Main.error(t);
    8182        }
    8283        }
    8384       
    84         public void add(Video video)
    85         {
    86                 try
    87                 {
     85        public void add(Video video) {
     86                try {
    8887                        EmbeddedMediaPlayer mp = mediaPlayerFactory.newEmbeddedMediaPlayer(fullScreenStrategy);
    8988                        video.player=mp;
    … …  
    9695                mp.playMedia(mediaPath);
    9796                //now fetching and playback starts automatically                       
    98                 }
    99                 catch (NoClassDefFoundError e)
    100         {
    101             System.err.println(tr("Unable to find JNA Java library!"));
    102         }
    103         catch (UnsatisfiedLinkError e)
    104         {
    105             System.err.println(tr("Unable to find native libvlc library!"));
     97                } catch (NoClassDefFoundError e) {
     98                    Main.error(tr("Unable to find JNA Java library!"));
     99        } catch (UnsatisfiedLinkError e) {
     100            Main.error(tr("Unable to find native libvlc library!"));
    106101        }
    107102        }
    … …  
    119114        }
    120115
    121         public void play()     
    122         {
    123                 if (singleVideoMode)
    124                 {
     116        public void play() {
     117                if (singleVideoMode) {
    125118                        lastAddedVideo.player.play();
    126                 }
    127                 else
    128                 {
     119                } else {
    129120                        for (Video video : videos) {
    130121                                video.player.play();
    131122                        }
    132123                }
    133                 System.out.println("abspielen");
    134124        }
    135125       
    136126        //toggles pause and play
    137         public void pause()
    138         {
    139                 if (singleVideoMode)
    140                 {
     127        public void pause() {
     128                if (singleVideoMode) {
    141129                        lastAddedVideo.player.pause();
    142                 }
    143                 else
    144                 {
     130                } else {
    145131                        for (Video video : videos) {
    146132                                video.player.pause();
    … …  
    159145        //jumps relative for ms in all videos
    160146        public void jumpFor(long ms) {
    161                 if (singleVideoMode)
    162                 {
     147                if (singleVideoMode) {
    163148                        long start=lastAddedVideo.player.getTime();
    164149                        lastAddedVideo.player.setTime(start+ms);
    165                 }
    166                 else
    167                 {
     150                } else {
    168151                        for (Video video : videos) {
    169152                                long start=video.player.getTime();
    … …  
    175158
    176159        //jumps in all videos to this absolute video time
    177         public void jumpTo(long msVideo)
    178         {
    179                 if (singleVideoMode)
    180                 {
     160        public void jumpTo(long msVideo) {
     161                if (singleVideoMode) {
    181162                        lastAddedVideo.player.setTime(msVideo);
    182                 }
    183                 else
    184                 {
     163                } else {
    185164                        for (Video video : videos) {
    186165                                video.player.setTime(msVideo);
    … …  
    191170                       
    192171        //TODO muss evtl. auf Rückgabe für alle Videos erweitert werden
    193         public long getVideoTime()
    194         {
     172        public long getVideoTime() {
    195173                return videos.get(0).player.getTime();
    196174        }
    197175       
    198176        //jumps in all videos to this absolute video time percentage
    199         public void jumpToPosition(int percent)
    200         {
     177        public void jumpToPosition(int percent) {
    201178                float position = ((float)percent/100f);
    202                 if (singleVideoMode)
    203                 {
     179                if (singleVideoMode) {
    204180                        lastAddedVideo.player.setPosition(position);
    205                 }
    206                 else
    207                 {
     181                } else {
    208182                        for (Video video : videos) {
    209183                                video.player.setPosition(position);
    … …  
    214188       
    215189        //TODO muss evtl. auf Rückgabe für alle Videos erweitert werden
    216         public int getPosition()
    217         {
     190        public int getPosition() {
    218191                return (int) (videos.get(0).player.getPosition()*100);
    219192        }       
    220193       
    221         public void setSpeed(int percent)
    222         {
    223                 if (singleVideoMode)
    224                 {
     194        public void setSpeed(int percent) {
     195                if (singleVideoMode) {
    225196                        lastAddedVideo.player.setRate((float)(percent/100f));
    226197                }
    … …  
    232203       
    233204        //TODO muss evtl. auf Rückgabe für alle Videos erweitert werden
    234         public int getSpeed()
    235         {
     205        public int getSpeed() {
    236206                return (int) (videos.get(0).player.getRate()*100);
    237207        }
    238208       
    239209        //returns if at least one video has subtitles
    240         public boolean hasSubtitles()
    241         {
     210        public boolean hasSubtitles() {
    242211                for (Video video : videos) {
    243212                        if (video.player.getSpuCount()>0) return true;
    … …  
    246215        }
    247216       
    248        
    249         public void setSubtitles (boolean enabled)
    250         {
    251                 if (enabled)
    252                 {
     217        public void setSubtitles (boolean enabled) {
     218                if (enabled) {
    253219                        //VLC uses a list of sub picture units
    254220                        for (Video video : videos) {
    255221                                video.player.setSpu(0);
    256222                        }
    257                 }
    258                 else
    259                 {
     223                } else {
    260224                        for (Video video : videos) {
    261225                                video.player.setSpu(-1);
    … …  
    263227                }
    264228        }
    265                
    266        
    267         public void setDeinterlacer (DeinterlaceMode deinterlacer)
    268         {
    269                 if (singleVideoMode)
    270                 {
     229       
     230        public void setDeinterlacer (DeinterlaceMode deinterlacer) {
     231                if (singleVideoMode) {
    271232                        lastAddedVideo.player.setDeinterlace(deinterlacer);
    272                 }
    273                 else
    274                 {
     233                } else {
    275234                        for (Video video : videos) {
    276235                                video.player.setDeinterlace(deinterlacer);
    … …  
    279238        }
    280239       
    281         public static String[] getDeinterlacers()
    282         {
     240        public static String[] getDeinterlacers() {
    283241                return deinterlacers;
    284242        }
    285243       
    286         public void mute()
    287         {
    288                 if (singleVideoMode)
    289                 {
     244        public void mute() {
     245                if (singleVideoMode) {
    290246                        lastAddedVideo.player.mute();
    291247                }
    … …  
    295251        }
    296252       
    297         public void unload()
    298         {
     253        public void unload() {
    299254                for (Video video : videos) {
    300255                        video.player.stop();
    … …  
    309264        public void addObserver(VideosObserver observer) {
    310265                observers.add(observer);
    311                
    312         }
    313 
    314         private void notifyObservers(VideoObserversEvents event)
    315         {
     266        }
     267
     268        private void notifyObservers(VideoObserversEvents event) {
    316269                for (VideosObserver observer : observers) {
    317270                        observer.update(event);
    … …  
    319272        }
    320273
    321         @Override
     274    @Override
    322275        public void backward(MediaPlayer arg0) { }
    323276/*
    … …  
    326279        }
    327280*/
    328         @Override
     281    @Override
    329282        public void error(MediaPlayer arg0) { }
    330283
    331         @Override
     284    @Override
    332285        public void finished(MediaPlayer arg0) { }
    333286
    334         @Override
     287    @Override
    335288        public void forward(MediaPlayer arg0) { }
    336289
    337         @Override
     290    @Override
    338291        public void lengthChanged(MediaPlayer arg0, long arg1) { }
    339292/*
    … …  
    348301        }
    349302*/
    350         @Override
     303    @Override
    351304        public void opening(MediaPlayer arg0) { }
    352305
    353         @Override
     306    @Override
    354307        public void pausableChanged(MediaPlayer arg0, int arg1) { }
    355308
    356         @Override
     309    @Override
    357310        public void paused(MediaPlayer arg0) { }
    358311
    359         @Override
     312    @Override
    360313        public void playing(MediaPlayer arg0) { }
    361314
    362         @Override
     315    @Override
    363316        public void positionChanged(MediaPlayer arg0, float arg1) { }
    364317
    365         @Override
     318    @Override
    366319        public void seekableChanged(MediaPlayer arg0, int arg1) { }
    367320
    368         @Override
     321    @Override
    369322        public void snapshotTaken(MediaPlayer arg0, String arg1) { }
    370323
    371         @Override
     324    @Override
    372325        public void stopped(MediaPlayer arg0) { }
    373326
    374         @Override
     327    @Override
    375328        public void timeChanged(MediaPlayer arg0, long arg1) { }
    376329
    377         @Override
     330    @Override
    378331        public void titleChanged(MediaPlayer arg0, int arg1) { }
    379332
    … …  
    388341        public void enableSingleVideoMode(boolean enabled) {
    389342                singleVideoMode = true;
    390                
    391343        }
    392344
    … …  
    394346    public void mediaChanged(MediaPlayer mediaPlayer, libvlc_media_t media, String mrl) {
    395347        // TODO Auto-generated method stub
    396        
    397348    }
    398349
    … …  
    400351    public void buffering(MediaPlayer mediaPlayer, float newCache) {
    401352        // TODO Auto-generated method stub
    402        
    403353    }
    404354
    … …  
    406356    public void videoOutput(MediaPlayer mediaPlayer, int newCount) {
    407357        // TODO Auto-generated method stub
    408        
    409358    }
    410359
    … …  
    412361    public void mediaMetaChanged(MediaPlayer mediaPlayer, int metaType) {
    413362        // TODO Auto-generated method stub
    414        
    415363    }
    416364
    … …  
    418366    public void mediaSubItemAdded(MediaPlayer mediaPlayer, libvlc_media_t subItem) {
    419367        // TODO Auto-generated method stub
    420        
    421368    }
    422369
    … …  
    424371    public void mediaDurationChanged(MediaPlayer mediaPlayer, long newDuration) {
    425372        // TODO Auto-generated method stub
    426        
    427373    }
    428374
    … …  
    430376    public void mediaParsedChanged(MediaPlayer mediaPlayer, int newStatus) {
    431377        // TODO Auto-generated method stub
    432        
    433378    }
    434379
    … …  
    436381    public void mediaFreed(MediaPlayer mediaPlayer) {
    437382        // TODO Auto-generated method stub
    438        
    439383    }
    440384
    … …  
    442386    public void mediaStateChanged(MediaPlayer mediaPlayer, int newState) {
    443387        // TODO Auto-generated method stub
    444        
    445388    }
    446389
    … …  
    448391    public void newMedia(MediaPlayer mediaPlayer) {
    449392        // TODO Auto-generated method stub
    450        
    451393    }
    452394
    … …  
    454396    public void subItemPlayed(MediaPlayer mediaPlayer, int subItemIndex) {
    455397        // TODO Auto-generated method stub
    456        
    457398    }
    458399
    … …  
    460401    public void subItemFinished(MediaPlayer mediaPlayer, int subItemIndex) {
    461402        // TODO Auto-generated method stub
    462        
    463403    }
    464404
    … …  
    466406    public void endOfSubItems(MediaPlayer mediaPlayer) {
    467407        // TODO Auto-generated method stub
    468        
    469     }
    470 
     408    }
    471409}
Note: See TracChangeset for help on using the changeset viewer.