[Magic-dev] Small bug with cifoutput variants

[R. Timothy Edwards]

Graham,

>    There is a small bug when using the variants option in the cifoutput section
> of the technology file (seen on version 7.4.44).
>    If a comma is used in the variant name, then the name is truncated when
> writing out the name in the comments in the cif header.
> 
> style pharosc variants (1um,vsclib200),(1um,rgalib200)
> 
> gives the following line in the cif header
> 
> ( @@style : pharosc(1um );
> 
> The unclosed parenthesis is probably illegal. It certainly causes some cif file
> readers to choke. Replacing the comma with a dash works:
> 
> style pharosc variants (1um-vsclib200),(1um-rgalib200)
> ( @@style : pharosc(1um-vsclib200) );

It's not really a bug, it's just that (as usual) I didn't consult anyone
on the syntax before I implemented the feature.  The variants are
always separated by commas, and as there are no other syntax rules,
parentheses are not parsed at all.  So "(((" is a valid variant.  Maybe
I should have used something less common as a delimiter, like a colon.
However, magic uses the comma as a delimiter for groups of layers, so
I followed that style.  At very least I will document the syntax.

					Regards,
					Tim

+--------------------------------+-------------------------------------+
| Dr. R. Timothy Edwards (Tim)   | email: tim at opencircuitdesign.com    |
| Open Circuit Design, Inc.	 | web:   http://opencircuitdesign.com |
| 22815 Timber Creek Lane        | phone: (301) 528-9364               |
| Clarksburg, MD 20871-4001      | cell:  (240) 401-0616               |
+--------------------------------+-------------------------------------+