kepka icon indicating copy to clipboard operation
kepka copied to clipboard

The Duplicating code...

Open leha-bot opened this issue 8 years ago • 6 comments

The Duplicating code never changes...

We should clean at most all code duplications. Feel free to add URLs to duplicated code chunks.

leha-bot avatar Dec 05 '17 22:12 leha-bot

CRC duplication (by @crackedmind)

https://github.com/procxx/tdesktop/blob/aca957595a0938345821994df85437b6fed574e2/Telegram/SourceFiles/core/utils.cpp#L387 https://github.com/procxx/tdesktop/blob/aca957595a0938345821994df85437b6fed574e2/Telegram/SourceFiles/codegen/emoji/generator.cpp#L127 https://github.com/procxx/tdesktop/blob/aca957595a0938345821994df85437b6fed574e2/Telegram/SourceFiles/codegen/style/generator.cpp#L44

leha-bot avatar Dec 05 '17 22:12 leha-bot

Long UI controller duplication logic (by @crackedmind)

Stanislav Ershov, [05.12.17 22:05] https://github.com/procxx/tdesktop/blob/aca957595a0938345821994df85437b6fed574e2/Telegram/SourceFiles/ui/widgets/input_fields.cpp#L2155 https://github.com/procxx/tdesktop/blob/aca957595a0938345821994df85437b6fed574e2/Telegram/SourceFiles/ui/widgets/input_fields.cpp#L2895

leha-bot avatar Dec 05 '17 22:12 leha-bot

media_player_cover.cpp media_player_widget.cpp media_clip_controller.cpp

all share the same code for bounding player positionMs

berkus avatar Dec 19 '17 22:12 berkus

core/utils.cpp implements sha and md5 functions despite dependency on openssl libs

berkus avatar Dec 20 '17 00:12 berkus

Search for case mtpc_messageEmpty - the code around it is repeated at least in 3 different files

berkus avatar Dec 20 '17 00:12 berkus

Version string generating.

https://github.com/procxx/kepka/blob/6ffc8d3905447a788ef139bd41ecc20f3c5797f2/Telegram/SourceFiles/apiwrap.cpp#L68

https://github.com/procxx/kepka/blob/6ffc8d3905447a788ef139bd41ecc20f3c5797f2/Telegram/SourceFiles/window/window_main_menu.cpp#L70

leha-bot avatar Feb 12 '18 22:02 leha-bot