anonym | Anmelden | Neues Konto anmelden | 2022-08-15 03:35 UTC |
Startseite | Übersicht | Probleme anzeigen | Änderungsprotokoll | Roadmap | Dokumentation | Konto |
Einfache Problemansicht anzeigen [ Zu Notizen wechseln ] | [ erweiterte Anzeige ] [ Problem-Historie ] [ Drucken ] | ||||||
ID | Kategorie | Auswirkung | Reproduzierbar | Meldungsdatum | Letzte Aktualisierung | ||
0000076 | [Mascot] general | kleinerer Fehler | nicht getestet | 2011-08-08 16:19 | 2012-08-29 20:17 | ||
Reporter | sigonnea | Anzeigestatus | öffentlich | ||||
Bearbeitung durch | xclerc | ||||||
Priorität | normal | Lösung | erledigt | ||||
Status | erledigt | Produktversion | 1.0-beta | ||||
Zusammenfassung | 0000076: No mandatory spaces around range operator | ||||||
Beschreibung | According to the example in Ocaml documentation (see section 7.4), range pattern does not have to be surrounded by whitespace. | ||||||
Zusätzliche Information |
Below is a small patch that corrects this: diff --git a/src/checks/typo_spacesAroundOperators.ml b/src/checks/typo_spacesAroundOperators.ml index 2d9046e..5ca5ac0 100644 --- a/src/checks/typo_spacesAroundOperators.ml +++ b/src/checks/typo_spacesAroundOperators.ml @@ -46,13 +46,13 @@ let run _ _ (_, tokens) _ report = (match s.[0] with | '-' | '~' -> Camlp4Utils.check_white "operator" s err prev - | '$' | '%' | '&' | '*' | '+' | '/' | '<' | '=' | '>' | (* '?' | *) '@' | '^' | '|' -> + | '$' | '%' | '&' | '*' | '+' | '/' | '<' | '=' | '>' | '@' | '^' | '|' -> Camlp4Utils.check_white "operator" s err prev; Camlp4Utils.check_first_white "operator" s err tl | ':' when len > 1 && s.[1] = ':' -> Camlp4Utils.check_white "operator" s err prev; Camlp4Utils.check_first_white "operator" s err tl - | '.' when len > 1 -> + | '.' when len > 1 && s <> ".." -> Camlp4Utils.check_white "operator" s err prev; Camlp4Utils.check_first_white "operator" s err tl | _ -> ()); |
||||||
Tags | Keine Tags zugeordnet. | ||||||
Angehängte Dateien | |||||||
|
Zu diesem Problem gibt es keine Notizen. |
Mantis 1.1.7[^] Copyright © 2000 - 2008 Mantis Group |