Author: jerome
Date: 2009-03-11 13:57:03 +0100 (Wed, 11 Mar 2009)
New Revision: 3951

Modified:
   
software_suite_v2/software/gadgets/tuxdroid-gadget-WMP/trunk/tuxdroid-gadget-WMP/src/com/kysoh/tuxdroid/Configuration.java
   
software_suite_v2/software/gadgets/tuxdroid-gadget-WMP/trunk/tuxdroid-gadget-WMP/src/com/kysoh/tuxdroid/MediaPlayerThread.java
Log:
* Fixed process listener bug.
* Fixed a bug : wmp was starting with the gadget path when no config was given.

Modified: 
software_suite_v2/software/gadgets/tuxdroid-gadget-WMP/trunk/tuxdroid-gadget-WMP/src/com/kysoh/tuxdroid/Configuration.java
===================================================================
--- 
software_suite_v2/software/gadgets/tuxdroid-gadget-WMP/trunk/tuxdroid-gadget-WMP/src/com/kysoh/tuxdroid/Configuration.java
  2009-03-11 12:35:16 UTC (rev 3950)
+++ 
software_suite_v2/software/gadgets/tuxdroid-gadget-WMP/trunk/tuxdroid-gadget-WMP/src/com/kysoh/tuxdroid/Configuration.java
  2009-03-11 12:57:03 UTC (rev 3951)
@@ -62,7 +62,14 @@
        }
        else
        {
-               args.add("\""+ dir.getAbsolutePath() +"\"");
+               if(!dir.getAbsolutePath().contains("tuxdroid-gadget-WMP"))
+               {
+                       args.add("\""+ dir.getAbsolutePath() +"\"");
+               }
+               else
+               {
+                       args.add("");
+               }
        }
        
        String[] arguments = new String[args.size()];

Modified: 
software_suite_v2/software/gadgets/tuxdroid-gadget-WMP/trunk/tuxdroid-gadget-WMP/src/com/kysoh/tuxdroid/MediaPlayerThread.java
===================================================================
--- 
software_suite_v2/software/gadgets/tuxdroid-gadget-WMP/trunk/tuxdroid-gadget-WMP/src/com/kysoh/tuxdroid/MediaPlayerThread.java
      2009-03-11 12:35:16 UTC (rev 3950)
+++ 
software_suite_v2/software/gadgets/tuxdroid-gadget-WMP/trunk/tuxdroid-gadget-WMP/src/com/kysoh/tuxdroid/MediaPlayerThread.java
      2009-03-11 12:57:03 UTC (rev 3951)
@@ -100,8 +100,8 @@
                
                try
                {                       
-                       //Starting media player.
                        Runtime.getRuntime().exec(command);
+
                        //checking if wmp is started by checking for his 
process name.
                        if(isWMPStarted())
                        {
@@ -177,36 +177,31 @@
           */
          public static boolean isWMPStarted()
          {
-                 String cmd = "tasklist /FI \"IMAGENAME eq wmplayer.exe\"";
-                 try 
-                 {
-                       byte count =0;
-                       //Total check is under one minute.
-                       while(count < 60)
+               String cmd = "tasklist /FI \"IMAGENAME eq wmplayer.exe\"";
+               //Total check is under one minute.
+               Process wmpRunning;
+               try 
+               {
+                       wmpRunning = Runtime.getRuntime().exec(cmd);
+                       StreamReader reader = new 
StreamReader(wmpRunning.getInputStream());
+                       reader.start();
+                       wmpRunning.waitFor();
+                       reader.join();
+                       String result = reader.getResult().trim();
+                       if(result.contains("wmplayer.exe"))
                        {
-                               Process wmpRunning = 
Runtime.getRuntime().exec(cmd);
-                               StreamReader reader = new 
StreamReader(wmpRunning.getInputStream());
-                               reader.start();
-                               wmpRunning.waitFor();
-                               reader.join();
-                               String result = reader.getResult().trim();
-                               if(result.contains("wmplayer.exe"))
-                               {
-                                       return true;
-                               }
-                               count++;
-                               Thread.sleep(1000);
+                               return true;
                        }
-                 } 
-                 catch (IOException e)
-                 {
+               } 
+               catch (IOException e) 
+               {
                        ;
-                 } 
-                 catch (InterruptedException e) 
-                 {
+               } 
+               catch (InterruptedException e) 
+               {
                        ;
                }
-                 return false;
+               return false;
          }
          
          


------------------------------------------------------------------------------
Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are
powering Web 2.0 with engaging, cross-platform capabilities. Quickly and
easily build your RIAs with Flex Builder, the Eclipse(TM)based development
software that enables intelligent coding and step-through debugging.
Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com
_______________________________________________
Tux-droid-svn mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/tux-droid-svn

Reply via email to