Move linear system export and make it safer.

[atgeirr]

Intended to remove unrelated export functionality from the linearization class, and avoid sprintf() which generates tons of warnings on clang.

• Remove the print...() functions that only wrote partial information. If this is needed it may be done by extracting from the export...() output instead.
• Move the export...() functions to a separate file. Keeping the top-level function though to assist future debugging, and the logic it contained relating to avoiding re-writing the sparsity pattern.
• Make the implementation safer by removing all use of sprintf().

[atgeirr]

@nrseman any comments?

[atgeirr]

jenkins build this please

[nrseman]

@nrseman any comments?

Looks like a reasonable solution to me. Did you verify that it works?

[nrseman]

@atgeirr, did you have a chance to verify that the export functionality still works?

[atgeirr]

@atgeirr, did you have a chance to verify that the export functionality still works?

Not yet, will try to get it done this week.

[atgeirr]

Ok, have made a few minor changes and verified that it writes the output (not written a tool to read it back in and check though). Taking it out of draft.

This is still very much a "tool for the debugging developer", as I have added no error checking for example, so it is easy to make it go kaboom (by passing a nonexisting directory for example), but I leave improving it to future interested parties.

[atgeirr]

jenkins build this please

[akva2]

As you point out yourself, this is not the end of the story, but it is a clear improvement so in she goes.

[nrseman]

Thank you gents. This utility is bread and butter for me.