What's wrong with my store procedure?

Question

HI, I can't figure out what's wrong with my stored procedure. I believe i have a "commit;" where it's not supposed to be. What i really want to acheive is save the record once it finds those keywords. Maybe my syntax is all wrong.
I am getting the following error msg. PLS-00103
create or replace
procedure sp_diag is
DECLARE
v_setting_key NUMBER(18);
v_parent NUMBER(22);
v_name VARCHAR2(100);
v_class_key NUMBER(22);
CURSOR c1 IS
select setting_key, setting_class_key,name
from cms2wire.setting_list;
CURSOR c2 IS
select setting_class_key, parent_class_key,name
from cms2wire.setting_class
where setting_class_key = v_parent;
BEGIN
OPEN c1;
LOOP
FETCH c1 into v_setting_key, v_parent, v_name;
EXIT when c1%NOTFOUND;
WHILE v_parent != null
LOOP
OPEN c2;
FETCH c2 into v_class_key, v_parent, v_name;
CLOSE c2;
IF v_name='Diag.'or v_name='Settings.' THEN
COMMIT;
END IF;
END LOOP;
END LOOP;
CLOSE c1;
END;
-------------------------------------------------------------------------------------

Answer 1

Now, the way I see it, if you want to save the information, then you need to put it some place, so your slow-by-slow stored procedure becomes something like
INSERT INTO some_table (column list)
  SELECT a.setting_key,
         a.setting_class_key,
         a.name
         b.setting_class_key,
         b.parent_class_key,
         b.name
    FROM cms2wire.setting_list  a,
         cms2wire.setting_class b
   WHERE a.setting_class_key = a.setting_class_key
     AND b.name in ('Diag.', 'Settings.');
COMMIT;does it not?