Ignore:
Timestamp:
10/17/09 02:05:35 (10 years ago)
Author:
Kaydeth_parpg
Message:

Ticket #2: Patch by Kaydeth. Updated the classes in local_loaders and utilities directories to correct some obvious discrepancies with our coding standard. comment[s:trac, t:2] fixes[s:trac, t:2]

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/game/utilities/agentXmlGen.py

    r317 r328  
    1414#   along with this program.  If not, see <http://www.gnu.org/licenses/>. 
    1515 
    16 import os, sys, getopt, logging, re 
     16import os 
     17import sys 
     18import getopt 
     19import logging 
     20import re 
    1721 
    1822log = logging.getLogger("agentxml") 
    1923log.setLevel(logging.DEBUG) 
    2024 
    21 def generateAgent(path, delay, xoff, yoff): 
     25def generateAgent(path, delay, x_off, y_off): 
    2226    """ Generates the agent XML file and creates animation 
    2327    XML files for found children.  
     
    2832    @type xoff: string 
    2933    @param xoff: X offset (used in animation.xml) 
    30     @type yoff: string 
    31     @param yoff: Y offset (used in animation.xml) 
     34    @type y_off: string 
     35    @param y_off: Y offset (used in animation.xml) 
    3236    @return True on success, False on failure 
    3337    """ 
     
    3943    log.debug("Opened " + os.path.join(path,name + ".xml")) 
    4044    f.write('<?fife type="object"?>\n') 
    41     f.write('<object id="%s" namespace="PARPG" blocking="1" static="0">\n' % (name,)) 
     45    f.write('<object id="%s" namespace="PARPG" blocking="1" static="0">\n' % \ 
     46            (name,)) 
    4247 
    4348    # Iterate the animations  
     
    4550    anims.sort() 
    4651    for anim in anims: 
    47         animpath=os.path.join(path,anim) 
    48         if os.path.isdir(animpath) and anim.find("svn")==-1: 
     52        anim_path=os.path.join(path,anim) 
     53        if os.path.isdir(anim_path) and anim.find("svn")==-1: 
    4954            log.debug("Found animation: %s", anim) 
    5055            f.write('\t<action id="' + anim +'">\n') 
    51             angles = os.listdir(animpath) 
     56            angles = os.listdir(anim_path) 
    5257            angles.sort() 
    5358            for angle in angles: 
    54                 anglepath=os.path.join(animpath,angle) 
    55                 if os.path.isdir(anglepath) and angle.find("svn")==-1: 
     59                angle_path=os.path.join(anim_path,angle) 
     60                if os.path.isdir(angle_path) and angle.find("svn")==-1: 
    5661                    log.debug("Found angle: %s", angle) 
    5762                    f.write('\t\t<animation source="' + anim + '/' + angle  
    5863                            + '/animation.xml" direction="'  
    5964                            + str(int(angle)) + '" />\n') 
    60                     if not generateAnimation(anglepath,delay,xoff,yoff): 
    61                         log.error("Failed to create animation for " + anum + " (angle: " + angle + ")") 
     65                    if not generateAnimation(angle_path,delay,xoff,y_off): 
     66                        log.error("Failed to create animation for " + anum\ 
     67                                   + " (angle: " + angle + ")") 
    6268            f.write('\t</action>\n') 
    6369    f.write('</object>\n') 
     
    7682    regex="[^0-9]*([0-9]*).*" 
    7783    try: 
    78         intX=int(re.sub(regex,"\\1",x)) 
     84        int_x = int(re.sub(regex,"\\1",x)) 
    7985    except: 
    80         intX=-1 
     86        int_x = -1 
    8187 
    8288    try: 
    83         intY=int(re.sub(regex,"\\1",y)) 
     89        int_y = int(re.sub(regex,"\\1",y)) 
    8490    except: 
    85         intY=-1 
     91        int_y = -1 
    8692     
    87     if intX>intY: 
     93    if int_x > int_y: 
    8894        return 1 
    89     elif intX<intY: 
     95    elif int_x < int_y: 
    9096        return -1; 
    9197     
    9298    return 0 
    9399 
    94 def generateAnimation(path, delay, xoff, yoff): 
     100def generateAnimation(path, delay, xoff, y_off): 
    95101    """ Generate animation.xml for a given path. 
    96102    @type path: string  
     
    98104    @type delay: string 
    99105    @param delay: Animation delay  
    100     @type xoff: string 
    101     @param xoff: X offset  
    102     @type yoff: string 
    103     @param yoff: Y offset  
     106    @type x_off: string 
     107    @param x_off: X offset  
     108    @type y_off: string 
     109    @param y_off: Y offset  
    104110    @return True on success, False on failure """ 
    105111 
    106     (tmp,angle)=os.path.split(os.path.abspath(path)) 
    107     (tmp,anim)=os.path.split(tmp) 
    108     (tmp,agent)=os.path.split(tmp) 
    109     id="%s:%s:%s"%(agent,anim,angle) 
     112    (tmp, angle) = os.path.split(os.path.abspath(path)) 
     113    (tmp, anim) = os.path.split(tmp) 
     114    (tmp, agent) = os.path.split(tmp) 
     115    id = "%s:%s:%s"%(agent, anim, angle) 
    110116    log.debug("Doing animation with id: %s" % (id)) 
    111117 
    112     images=os.listdir(path) 
     118    images = os.listdir(path) 
    113119    # Get those with the correct extension 
    114     imagesfiltered=[] 
     120    images_filtered = [] 
    115121    for image in images: 
    116122        if image.endswith(".png"): 
    117             imagesfiltered.append(image) 
     123            images_filtered.append(image) 
    118124 
    119125 
     
    121127    log.debug("Opened: " + os.path.join("animation.xml")) 
    122128    f.write('<animation delay="' + delay + '" namespace="PAPRG" id="' + id  
    123             + '" x_offset="' + xoff  
    124             + '" y_offset="' + yoff + '">\n') 
     129            + '" x_offset="' + x_off  
     130            + '" y_offset="' + y_off + '">\n') 
    125131 
    126132    # Sort them 
    127     imagesfiltered.sort(cmp=imageCompare) 
    128     for image in imagesfiltered: 
     133    images_filtered.sort(cmp=imageCompare) 
     134    for image in images_filtered: 
    129135        log.debug("Found image: %s" % image) 
    130136        f.write('\t<frame source="' + image + '" />\n') 
     
    157163        sys.exit(2) 
    158164 
    159     agentpath="." 
    160     delay="100" 
    161     x_offset="0" 
    162     y_offset="0" 
    163     verbose=False 
     165    agent_path = "." 
     166    delay = "100" 
     167    x_offset = "0" 
     168    y_offset = "0" 
     169    verbose = False 
    164170 
    165171    ch = logging.StreamHandler() 
     
    168174    for opt, arg in opts:         
    169175        if opt in ("-p","--path"): 
    170             agentpath=path 
     176            agent_path = path 
    171177        elif opt in ("-d","--delay"): 
    172             delay=arg 
     178            delay = arg 
    173179        elif opt in ("-x","--x_offset"): 
    174             x_offset=arg 
     180            x_offset = arg 
    175181        elif opt in ("-y","--y_offset"): 
    176             y_offset=arg 
     182            y_offset = arg 
    177183        elif opt in ("-v","--verbose"): 
    178184            ch.setLevel(logging.DEBUG) 
     
    185191    log.addHandler(ch) 
    186192 
    187     log.debug("Going to generate animation for %s" %(agentpath)) 
    188     generateAgent(agentpath, delay, x_offset, y_offset) 
     193    log.debug("Going to generate animation for %s" % (agent_path)) 
     194    generateAgent(agent_path, delay, x_offset, y_offset) 
    189195    log.info("Done"); 
    190196 
Note: See TracChangeset for help on using the changeset viewer.