Opened 10 years ago
Closed 8 years ago
#241 closed task (fixed)
Use built-in .Net methods
Reported by: | ibboard | Owned by: | |
---|---|---|---|
Priority: | trivial | Milestone: | |
Component: | WarFoundry-API | Version: | Trunk |
Keywords: | codingstandards dotnet papercut | Cc: | |
Blocked By: | Blocking: |
Description
(Via Twitter)
.Net seemingly has quite a collection of extra methods for common operations (e.g. joining file names to file paths). These may be optimised, but will certainly be more thoroughly tested. A list of suggested alternatives is here.
Code blocks 5 (combining paths using System.IO.Path), 6 (getting file extension using System.IO.Path) and 13 (using File.WriteAllText() to easily write a text file) are the most likely to occur in the code, although 1, 9 and 14 may also turn up in places.
Change History (5)
comment:1 Changed 10 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 10 years ago by
Keywords: | papercut added |
---|---|
Priority: | major → trivial |
comment:3 Changed 9 years ago by
Component: | General/Unknown → WarFoundry-API |
---|
comment:5 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | confirmed → closed |
r966 fixed all the obvious ones
Note: See
TracTickets for help on using
tickets.
Most important part for this ticket is the API.