revise code, 4 me! Options

[capt_insane]

can ya'll help me to c if there is anything wrong with this code!

<!-- begin code provided by createblog.com -->
<!-- begin code provided by createblog.com -->
Code:

<script language="javascript">
//chatterbox screen name tracker - xanga.com/add_ons
function getXName() {
for (var i = 0; i < document.links.length; i++) {
if (document.links[i].href.substring(0,38)=="http://www.xanga.com/private/home.aspx") {
var link = document.links[i];
var text = document.all ? link.innerText : link.text ? link.text : '';
return text;
break;
}}
return null;
}

function getPopLink() {
for (var i = 0; i < document.links.length; i++) {
if (document.links[i].id=='chatterpopup')
return document.links[i]; break;
} return null; }

xName = getXName();
if (xName != null && chatterbox == 'ok') {
nameBox = document.forms['chatterform'].elements['name'];
nameBox.value=xName;
chatter_name = xName;
urlBox = document.forms['chatterform'].elements['uurl'];
if (urlBox) {
urlBox.value='http://www.xanga.com/home.aspx?user=' + xName;
}

} else
if (xName == null && chatterbox == 'ok') {
nameBox = document.forms['chatterform'].elements['name'];
nameBox.value='Log in';
nameBox.disabled=true;


msgBox = document.forms['chatterform'].elements['msg'];
msgBox.value='to post';
msgBox.disabled=true;

postBtn =
document.forms['chatterform'].elements['post'];
postBtn.disabled = true;

urlBox = document.forms['chatterform'].elements['uurl'];
if (urlBox) {
urlBox.value='to Xanga';
urlBox.disabled=true;
}

popLink = getPopLink();
if (popLink != null) {
popLink.href = popLink.href + '&nopost=Log+in!!'; }

}
</script>

<!-- end code provided by createblog.com -->
<!-- end code provided by createblog.com -->

is there sappose to be 2 begin and end code thingesss?