View Issue Details

IDProjectCategoryView StatusLast Update
0038312pas2jsrtlpublic2021-01-23 11:41
Reporterhenrique Assigned ToMichael Van Canneyt  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionno change required 
PlatformPas2JsOSWindows 
Summary0038312: Just remove some declarations
DescriptionI just removed redundant claims
TagsNo tags attached.
Fixed in Revision
Attached Files

Activities

henrique

2021-01-05 15:05

reporter  

0001-Removido-declara-o-redundante.patch (1,015 bytes)   
From 18e2061c9a1d4d20fb4ba04a898badafa265d848 Mon Sep 17 00:00:00 2001
From: Henrique Gottardi Werlang <henriquewerlang@hotmail.com>
Date: Mon, 4 Jan 2021 09:00:24 -0300
Subject: [PATCH] =?UTF-8?q?Removido=20declara=C3=A7=C3=A3o=20redundante.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 packages/rtl/classes.pas | 2 --
 1 file changed, 2 deletions(-)

diff --git a/packages/rtl/classes.pas b/packages/rtl/classes.pas
index 4277005..d7990dc 100644
--- a/packages/rtl/classes.pas
+++ b/packages/rtl/classes.pas
@@ -49,8 +49,6 @@ type
   TListAssignOp = (laCopy, laAnd, laOr, laXor, laSrcUnique, laDestUnique);
   TListSortCompare = function(Item1, Item2: JSValue): Integer;
   TListSortCompareFunc = reference to function (Item1, Item2: JSValue): Integer;
-  TListCallback = Types.TListCallback;
-  TListStaticCallback = Types.TListStaticCallback;
   TAlignment = (taLeftJustify, taRightJustify, taCenter);
 
   // Forward class definitions
-- 
2.29.2.windows.2

Michael Van Canneyt

2021-01-23 11:41

administrator   ~0128510

The type aliases are there for backwards compatibility.
If someone refers to a type with a fully qualified name or he does not have the types unit included and uses this identifier, then he will get an error if you remove this.
The FPC codebase has the same aliases for the same reason.

Issue History

Date Modified Username Field Change
2021-01-05 15:05 henrique New Issue
2021-01-05 15:05 henrique File Added: 0001-Removido-declara-o-redundante.patch
2021-01-05 15:35 Mattias Gaertner Assigned To => Michael Van Canneyt
2021-01-05 15:35 Mattias Gaertner Status new => assigned
2021-01-23 11:41 Michael Van Canneyt Status assigned => resolved
2021-01-23 11:41 Michael Van Canneyt Resolution open => no change required
2021-01-23 11:41 Michael Van Canneyt Note Added: 0128510