Anonymous | Login | Signup for a new account | 2021-01-27 23:11 CET | ![]() |
Main | My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0000143 | YaCy | [All Projects] General | public | 2012-01-25 21:50 | 2015-02-19 03:18 | ||||
Reporter | Quix0r | ||||||||
Assigned To | |||||||||
Priority | low | Severity | trivial | Reproducibility | always | ||||
Status | resolved | Resolution | fixed | ||||||
ETA | none | ||||||||
Platform | Debian Unstable | OS | Linux | OS Version | 2.6.37 | ||||
Product Version | YaCy 1.0 | ||||||||
Target Version | Fixed in Version | YaCy 1.0 | |||||||
Summary | 0000143: Blacklist class: Rewritte all BLACKLIST_FOO to enum | ||||||||
Description | The enum may consume lesser memory as a bunch of final, static strings + a HashSet (BLACKLIST_TYPES) and is also a better type-hint in method signatures than a String. | ||||||||
Tags | code style, enum, HashSet, String, type-hint | ||||||||
Attached Files | |||||||||
![]() |
|
(0000385) Quix0r (updater) 2012-05-30 16:18 |
Ist mit commit https://gitorious.org/~quix0r/yacy/quix0rs-rc1/commit/8d8a63f9e76cc70aada0b9c7d67351ea6989629e [^] eingebaut. |
(0000386) Quix0r (updater) 2012-05-30 16:43 edited on: 2012-05-30 17:03 |
Mist, die enums sind alle gross geschrieben, ich wollte sie wie alle anderen enums auch gross schreiben und nicht auf einmal klein, um ein einheitliches Bild zu bekommen. Dadurch sind jetzt alle Blacklists deaktiviert und werden nicht beruecksichtigt. :( EDIT 1: Dazu muesste toString() ueberschrieben werden: @Override public final String toString () { return super.toString().toLowerCase(); } So sollte es dann wieder funktionieren, ohne wenig Code anzufassen und ohne alle Leute zum re-aktivieren der Blacklists auffordern zu muessen. :) EDIT 2: Ist hiermit eingebaut: https://gitorious.org/~quix0r/yacy/quix0rs-rc1/commit/9fef3c0e0952ce3ae4207ffb9ee72ff10144bf9e [^] Ich sollte mal mehr testen. :( |
(0000392) Quix0r (updater) 2012-06-04 17:11 |
Commit https://gitorious.org/~quix0r/yacy/quix0rs-rc1/commit/2900fde84ac8bab65ed84bb3f062755db48690e0 [^] enthaelt nun abermals die noetigen Aenderungen fuer das enum. Commit https://gitorious.org/~quix0r/yacy/quix0rs-rc1/commit/2c4ff32c86e3d849ad49c5e6bd41baf8cbe2fc7c [^] konvertiert nur Tabs in Leerschritte. :( |
(0000393) Quix0r (updater) 2012-06-11 20:55 |
Okay, ich musste nochmals Aenderungen vornehmen (final und @Override viel). Das meiste aber habe ich auch bei mir uebernommen. |
(0000408) Quix0r (updater) 2012-07-19 23:46 |
Ich denke, soweit ist das aufgenommen. Fehlt halt nur noch die Umstellung Map->ConcurrentMap, da Map eine allgemeine (generic) Map ist und ConcurrentMap die ConcurrentHashMap genauer umschreibt. Dies ist generell so Regeln, dass man nicht ein generic Interface, sondern eines nimmt, dass das betreffende Objekt am dichtesten umschreibt. Z.B. erlaubt Map auch TreeMap oder LinkedHashMap, beides ist aber keines falls "kompatibel" zu HashMap. |
![]() |
|||
Date Modified | Username | Field | Change |
2012-01-25 21:50 | Quix0r | New Issue | |
2012-01-25 21:50 | Quix0r | Tag Attached: code style | |
2012-01-25 21:50 | Quix0r | Tag Attached: enum | |
2012-01-25 21:50 | Quix0r | Tag Attached: HashSet | |
2012-01-25 21:50 | Quix0r | Tag Attached: String | |
2012-01-25 21:50 | Quix0r | Tag Attached: type-hint | |
2012-05-30 16:18 | Quix0r | Note Added: 0000385 | |
2012-05-30 16:19 | Quix0r | Resolution | open => fixed |
2012-05-30 16:43 | Quix0r | Note Added: 0000386 | |
2012-05-30 16:55 | Quix0r | Note Added: 0000387 | |
2012-05-30 16:56 | Quix0r | Note Deleted: 0000387 | |
2012-05-30 16:56 | Quix0r | Note Edited: 0000386 | View Revisions |
2012-05-30 16:59 | Quix0r | Note Edited: 0000386 | View Revisions |
2012-05-30 17:03 | Quix0r | Note Edited: 0000386 | View Revisions |
2012-06-04 17:11 | Quix0r | Note Added: 0000392 | |
2012-06-11 20:55 | Quix0r | Note Added: 0000393 | |
2012-07-19 23:46 | Quix0r | Note Added: 0000408 | |
2012-07-19 23:49 | Quix0r | Fixed in Version | => YaCy 1.0 |
2015-02-19 03:18 | BuBu | Status | new => resolved |
Copyright © 2000 - 2021 MantisBT Team |