tobin / Elektrotickle

Linear circuit simulation in Matlab
5 stars 1 forks source link

Accept commands in uppercase as LISO does #3

Open cwipf opened 10 years ago

cwipf commented 10 years ago

This little patch fixes it for me:

diff --git a/load_liso.m b/load_liso.m
index b323f69..71420fa 100644
--- a/load_liso.m
+++ b/load_liso.m
@@ -36,7 +36,7 @@ while true
     args = regexp(s, '\s+', 'split');

-    switch args{1}
+    switch lower(args{1})
         case 'r'     % r name value node1 node2 (resistor)
             name = args{2};
             value = circuit.parse_value(args{3});
@@ -80,7 +80,7 @@ while true
             startfreq = circuit.parse_value(args{3});
             stopfreq = circuit.parse_value(args{4});
             steps = circuit.parse_value(args{5});
-            switch args{2}
+            switch lower(args{2})
                 case 'lin'
                     f = linspace(startfreq, stopfreq, steps);
                 case 'log'
tobin commented 10 years ago

Are component and node names case-sensitive in LISO?

cwipf commented 10 years ago

Two component names differing only by case -- seems to work but issues a warning "name is already used, this may cause confusion" Two node names differing only by case -- issues a warning "several pointers to same location" and gives the wrong result