[MIT6.828]LAB1输入输出TAB字符的若干问题及总结

LAB1给代码中的'\t'的处理很简陋，直接把'\t'替换为5个空格，这和我们平时习惯有很大不同，所以要改造一下。

1、在kern/console.c cga_putc()函数中，可以看到'\t'case分支，把这个分支替换为如下代码:

        case '\t':
                do{
                        cons_putc(' ');
                }while(crt_pos%TAB_SIZE != 0);
                break;

其中的TAB_SIZE我将其定义在kern/console.h，尺寸为8：
#define TAB_SIZE 8
这样重新编译，用输出一些TAB字符，其行为已正常。

2、发现在系统终端下输入TAB是被禁止的，但是我想把TAB键作为制表符输入，所以要修改输入的部分，经过定位，发现是在lib/readline.c readline()函数中处理的输入，于是把22行的:

} else if ( c >= ' ' && i < BUFLEN-1) {

修改为：

} else if ( ((c >= ' ')||(c == '\t')) && i < BUFLEN-1) {

保存后重新编译，可以正常输入TAB字符。

3、又发现输入TAB后的退格处理不正常，一个TAB应该一次性退格，但是目前的退格仅仅是退一个空格，这不是我们想要的功能，发现对退格的处理仍然是在kern/console.c cga_putc()函数中。
思考：cga_putc()要怎么才能知道退格的是一个空格还是一个制表符？
参考步骤1，我们都是输出的' '，在显示缓冲区里面是不能区分，这必然要求我们用一个缓冲来保存以前处理的字符串，而且TAB在不同位置下长度不固定，也这也要求我们必须要记录其输出长度。对于缓冲长度要取多少，肯定要大于等于lib/readline.c readline()函数中缓冲区的长度，至于上限，就是个见仁见智的问题。
cga_putc()代码修改如下：

//mod by davelv @ 2010-11-02
//add the facility to display and clear '\t'
static void cga_putc(int c)
{
	static int line_buf[CRT_SIZE];//line buffer = 80*25,high_word:char's length,low_word:attribute &  value
	static int line_pos;//line buffer's current postion
	int i;//i as a temp var;
	// if no attribute given, then use black on white
	if (!(c & ~0xFF))
		c |= 0x0700;
	switch (c & 0xff) {
	case '\b':
		if ((crt_pos > 0)&&(line_pos > 0)) {//not only crt_pos but also line_pos
			i = line_buf[--line_pos]>>16;//get char's length
			do{
				crt_buf[--crt_pos] = (c & ~0xff) | ' ';//clear last char
			}while(((line_buf[line_pos]&0xff) == '\t')&&( --i));//loop when char '\t' hasn't been clear all;
		}
		break;
	case '\n':
		crt_pos += CRT_COLS;
		/* fallthru */
	case '\r':
		crt_pos -= (crt_pos % CRT_COLS);
		line_pos = 0;//when '\r' or '\n',clear buf
		break;
	case '\t':
		i = 0;//TAB length counter
		do{
			crt_buf[crt_pos++]=(c & ~0xff) | ' ';//display one blank
			i++;//length++
		}while(crt_pos%TAB_SIZE != 0);
		line_buf[line_pos++] = c|(i<<16) ;//save length&value
		break;
	default:
		line_buf[line_pos++] = c;	//save new char
		crt_buf[crt_pos++] = c;		/* write the character */
		break;
	}
	// What is the purpose of this?
	if (crt_pos >= CRT_SIZE) {
		memmove(crt_buf, crt_buf + CRT_COLS, (CRT_SIZE - CRT_COLS) * sizeof(uint16_t));
		for (i = CRT_SIZE - CRT_COLS; i < CRT_SIZE; i++)
			crt_buf[i] = 0x0700 | ' ';
		crt_pos -= CRT_COLS;
	}
	/* move that little blinky thing */
	outb(addr_6845, 14);
	outb(addr_6845 + 1, crt_pos >> 8);
	outb(addr_6845, 15);
	outb(addr_6845 + 1, crt_pos);
}

保存，编译，运行，无论是输入TAB还是删除TAB均已变得正常。

4、新的问题，我暂不说个人看法，先供大家讨论和思考：
（1）据我们所知，TAB的尺寸是可以由软件定制的，但是我们这个LAB只有内核，所以直接写死到驱动里面，但哪些是应该由驱动做的，哪些是由用户程序自定义的，这是OS设计者必须划分清楚的，在驱动里去做TAB，还有我新加的行缓冲，这样是否合理，应该怎样改进或者拆分。
（2）我这里把驱动里面的行缓冲定义为一屏字符，但是如果只输出TAB的话，其实是八屏，但是一旦屏幕字符刷到上面去以后，就不能再回来。如何实现整屏缓冲，缓冲要做到哪里合适。

不知道新的LAB中会不会添加新的代码，如果添加新的代码正好把这部分做了的话，倒是可以参考下。