ViewString does not correctly escape quotes inside strings

[nathancarter]

Observed behaviour

ViewString does not correctly escape quotes inside strings.

gap> Print(ViewString("fo\"o"));
"fo"o"

Expected behaviour

The same behavior one gets from directly evaluating the string in the console.

gap> "fo\"o";
"fo\"o"

Copy and paste GAP banner (to tell us about your setup)

 ┌───────┐   GAP 4.9.2 of 04-Jul-2018
 │  GAP  │   https://www.gap-system.org
 └───────┘   Architecture: x86_64-apple-darwin17.7.0-default64
 Configuration:  gmp 6.1.2
 Loading the library and packages ...
 Packages:   AClib 1.3, Alnuth 3.1.0, AtlasRep 1.5.1, AutPGrp 1.9, Browse 1.8.8, CRISP 1.4.4, Cryst 4.1.17, CrystCat 1.1.8, CTblLib 1.2.2,
             FactInt 1.6.2, FGA 1.4.0, GAPDoc 1.6.1, IO 4.5.1, IRREDSOL 1.4, LAGUNA 3.9.0, Polenta 1.3.8, Polycyclic 2.14, PrimGrp 3.3.1,
             RadiRoot 2.8, ResClasses 4.7.1, SmallGrp 1.3, Sophus 1.24, SpinSym 1.5, TomLib 1.2.6, TransGrp 2.0.2, utils 0.54

[stevelinton]

That's a clear bug in the ViewString method for lists. If it is going to put " around the string it should escape any necessary characters in the string.