james-d-mitchell / gaplint

A linter for GAP (https://www.gap-system.org/)
GNU General Public License v3.0
7 stars 4 forks source link

Handling long strings in test files #2

Closed wilfwilson closed 7 years ago

wilfwilson commented 7 years ago

Chris added the following test to a test file in the Digraphs package:

gap> gr := DigraphFromDiSparse6String(
> ".~?CI_A?WA_M@G@_G@gB?]@?G_SAWA?Y@oJ__BGH?uA_M_IAoO_oCWL@IB_R_{DGB?mB?U_sDwM@a\
> BoX_KCGP@WEwQ@[FGR@_FWS@cFgT@gFwB@A?oO_KCGZACGwZAGGw[AUFOcAYF_f`{IG_Ae@?U`[IwW\
> AqEOl`gJgC@mF?jBAFOkBEF_lBIFomBMG?nBQ@?ZAE@?ZAI@?ZAMH?oBWMGdB?LowaWKOya[K_xBmI\
> ?rBqIOsBuI_tBy@?`AMH?uCA@?aaOHOuB[OgCAIHOvCQ@?baOM@?CYMP@C]HOwCYMOyCKPpHbgPPHb\
> kQw{CgRG|C_RW}CyNpN_SIwkDEJPQawSwnDQ@OobCSPUbGS`VbKSpWbOT@XbSTPY_SK?u_SK?v_SK?\
> wbcT`[DyM`UDsVgzD[WG{D_Vp`bsUPabwU`bb{Upc_SL`?bcOP[EY@OvCIMOyCKV@\\Ea@OvCQM`DD\
> sYgDB_PgxC[V`kc_VpfEuM`HDwZHID{W@hEsZxJE?YpncoWPpcsW`oFIR`bEw[xNEO\\HOES\\WDC?\
> Ph@C[X`vc_Xpw_SOhBEWY@yc_Q`fEc]wDCQH@DE_Y`|cgQphEk^gDCQPPiGAQpjGE@PEc[Z@vGMQ@l\
> F_`HmFc`XHEo_xIEsZqCG]Z`oFo`QGckZqFf?[P~G_ahpGGahqGq[qJGu\\AHGy\\QEG}\\aO_G@gF\
> @?cgGAKcwHCWdGIGKdWA_KCAV`CeGQACeWRAOegSC[ewTGOfG`AGeGBH]GaWH}@?bHaH?dC?Oa]IEH\
> Q_IEHaba[gaca_OQdacQAeag`QfcCOqa_OeA^aCO`CI?ihBCSgabIm@A^cOgAlcSgqm_SPa`c[QPvF\
> ggaoc_Q`wFkiQpccgqocgQqkJCkxJI{kxKJURQsJYRaqJ]RpxJaSAEJe?qQaGcqzaSdA{ccdQ|g[da\
> }hcigQCGoHBHkoXIHooiGHsoy^i?ialKU@Q`IiH@yFsgqjK]grEK]hBHa[oRGKiI@zISobJfohrKgc\
> iBL_OmxCJorxDJssHJJwsYIJ{sgDIgjZ?LQC`|LU^aZKCth~KKtyJKOuGDK}_BOLi_RPLm_bQLqbBR\
> Lu@gPHKvweJswHKLGwYLLwwgDIuCaVMAHqZH{wRdcsjRBMGxiMLcwrcM]GP|G?jbELQ_AmMQjRDkWt\
> BcMmxBja_fa_KSxhMI[iqmKWxrjMybrMM_yRiMozRncC]p}IoqH}GCganMeOamKYOqnK_qRhNMjbEM\
> mO`|MgzBtmgzRti{qRkNUyzkMs|Rxms}XNIcjAnM{{rtNe_QnMg|YOJg{BpNG|BuN[}ByNk~B|fc^A\
> qfo^qqJO{XzNG|H~GGlAtNG|H}K_|bwNu^RvNi_atN_~ZvN_}b|n[}bznmcR}N|?C@OH?sCOT@cFOb\
> AiQ@ECgR`QDWaaSOgea]IGhaiOxCcUQPycg]xJcqRXMc}SH{fsgx}f}_I@gI`yGgeaiJgqbYMg}cIP\
> heeiZhqfYciUhifiajinjMlItjYlywjemj?kEojBkQqZIkmrJLkytZUl]uJXmkyjrNI}Rv");

Gaplint complains WARNING in tst/standard/prop.tst:1077 unmatched quote!.

However, I think that this usage should be allowed.

james-d-mitchell commented 7 years ago

Fair enough, I'll try to fix that today

james-d-mitchell commented 7 years ago

@wilfwilson I think this is now resolved, can you double check and let me know?

wilfwilson commented 7 years ago

Seems good!