view Android/java/org/madbutterfly/paint.java @ 887:8a2d676d9fa3 abs_n_rel_center

Fix bug of clearing COF_TEMP_MARK. draw_shapes_in_dirty_areas() mark COF_TEMP_MARK on coords and clear them before leaving. But, it always clear the same one coord for several times and keep others untouched. It is a logical error. It can be avoided by avoiding to use the variable at mutliple place for different purpose in a function, and compiler will check it out by yelling that the variable is used before assigning a value.
author Thinker K.F. Li <thinker@codemud.net>
date Sun, 26 Sep 2010 14:07:05 +0800
parents 2d7df44e6299
children
line wrap: on
line source

package org.madbutterfly;

import java.util.ArrayList;

public class paint {
    protected int addr;
    redraw_man rdman;
    ArrayList members;

    public paint(redraw_man rdman, int addr) {
	this.addr = addr;
	this.rdman = rdman;
	members = new ArrayList();
    }

    protected void invalid() {
	addr = 0;
    }

    protected void finalize() {
	if(addr != 0)
	    _jni.rdman_paint_free(rdman._rdman_addr, addr);
    }

    public void stroke(shape sh) {
	no_stroke(sh);
	
	if(sh.fill != this)
	    members.add(sh);

	sh.stroke = this;
	
	_jni.rdman_paint_stroke(rdman._rdman_addr, addr, sh.addr);
    }
    
    public void fill(shape sh) {
	no_fill(sh);
	
	if(sh.stroke != this)
	    members.add(sh);

	sh.fill = this;
	
	_jni.rdman_paint_fill(rdman._rdman_addr, addr, sh.addr);
    }

    public boolean is_empty() {
	return members.size() == 0;
    }

    public static void no_stroke(shape sh) {
	int i;
	
	if(sh.stroke != null && sh.stroke != sh.fill) {
	    i = sh.stroke.members.indexOf(sh);
	    sh.stroke.members.remove(i);
	}
	sh.stroke = null;
    }

    public static void no_fill(shape sh) {
	int i;
	
	if(sh.fill != null && sh.stroke != sh.fill) {
	    i = sh.stroke.members.indexOf(sh);
	    sh.stroke.members.remove(i);
	}
	sh.fill = null;
    }
}