Ticket #1382 (new defect)

Opened 2 years ago

Last modified 5 days ago

Colors and Bug Fix

Reported by: rahly@terrygoodkind.net Owned by: rakshasa
Priority: lowest Milestone:
Component: rtorrent Version:
Severity: trivial Keywords:
Cc:

Description

Found a bug in rtorrent (at least with remoting with putty) where if the canvas isn't in multiples of 3's it would mess up the bottom of the of the "window" by combining lines.

Also hardcoded colors in, Bright Green text for finished torrents, and bright red for "open" torrents, which are torrents that have been started but not finished. This is not the same as running torrents.

Patch is included.

Attachments

rtorrent-0.7.9-canvas-fix.patch (1.9 KB) - added by rahly@terrygoodkind.net 2 years ago.
Fix + Colors
rtorrent-0.7.9-canvas-fix-2.patch (10.1 KB) - added by rahly@terrygoodkind.net 2 years ago.
rtorrent-0.8.4-SVN-canvas-fix.patch (10.2 KB) - added by rahly 21 months ago.
updated patch for SVN
rtorr.diff (2.2 KB) - added by sleip 16 months ago.
patch to use reverse video on the main window download list instead of "*" for selected item
rtorrent-0.8.6-canvas-fix.patch (9.0 KB) - added by sleip 8 months ago.
updated rahlys patch to 0.8.6
canvas-color_1163.patch (9.9 KB) - added by hyper_ch 5 months ago.
Canvas Color Patch designed for rev. 1163

Change History

Changed 2 years ago by rahly@terrygoodkind.net

Fix + Colors

Changed 2 years ago by rahly@terrygoodkind.net

  Changed 2 years ago by rahly@terrygoodkind.net

Added new patch for customability. Now you can specify the color you wish active torrents and done torrents to be in the .rtorrent.rc file

new entries include

done_fg_color done_bg_color active_fg_color active_bg_color

using a value of -1 for any color will use the terminals default color

Colors are the same for curses

0 = black 1 = red 2 = green 3 = yellow 4 = blue 5 = magenta 6 = cyan 7 = white

patch also includes previous fix

  Changed 2 years ago by rahly@terrygoodkind.net

Oh yeah, also set the list so that the torrent title is bold if

A) done and upload rate > 0 B) active and download rate > 0

otherwise if there isn't activity, the torrent appears without bold

  Changed 2 years ago by rahly@terrygoodkind.net

patch works for current SVN as well

  Changed 2 years ago by Klav

Works great (using SVN), thanks!

  Changed 2 years ago by anonymous

i'm loving it and have been using it since it was posted here :) (with 0.7.9)

thanks!

  Changed 23 months ago by anonymous

works with 0.8.3

Changed 21 months ago by rahly

updated patch for SVN

  Changed 18 months ago by anonymous

how to apply this patch on 0.8.4 rev 1087

  Changed 18 months ago by anonymous

Download the patch, save it in the rtorrent source directory

Quote: patch -p1 < rtorrent-0.8.4-SVN-canvas-fix.patch

The result should be

Quote: patching file src/command_network.cc Hunk #1 succeeded at 368 (offset 40 lines). patching file src/display/canvas.cc Hunk #1 succeeded at 91 (offset 14 lines). patching file src/display/window_download_list.cc patching file src/display/window_download_list.h patching file src/ui/download_list.cc Hunk #1 succeeded at 136 with fuzz 2 (offset 16 lines). patching file src/ui/download_list.h patching file src/ui/element_download_list.h Hunk #1 succeeded at 59 with fuzz 2. patching file src/ui/root.cc Hunk #1 succeeded at 43 (offset 1 line). Hunk #2 succeeded at 65 (offset 1 line). Hunk #3 succeeded at 101 (offset 1 line). Hunk #4 succeeded at 246 with fuzz 2 (offset 1 line). patching file src/ui/root.h Hunk #1 succeeded at 81 (offset 2 lines). Hunk #2 succeeded at 110 (offset 2 lines).

If it is, recompile. If it's not, you've probably screwed up your sources, redownload them.

  Changed 18 months ago by anonymous

is there a way not only to hightlight active torrents?

  Changed 16 months ago by rahly

its possible to highlight other things, but why would you want to highlight non active torrents? what would that criteria entail?

Changed 16 months ago by sleip

patch to use reverse video on the main window download list instead of "*" for selected item

  Changed 16 months ago by sleip

that reverse video patch is like #58 but works only with rahlys patch.

  Changed 14 months ago by anonymous

Could you attach a screenshoot?

  Changed 14 months ago by rahly

Quick screenshot, showing the 5 color states.

1. closed
2. Seeding(have all the file) and actively uploading
3. not downloaded and stalled(not actively downloading)
4. actively downloading
5. Seeding by not actively uploading.
http://dl748.com/rtorrent.png

follow-up: ↓ 15   Changed 14 months ago by anonymous

Thanks!

in reply to: ↑ 14   Changed 14 months ago by anonymous

Replying to anonymous:

Thanks!

Remember though, the colors themselves, are configurable.

  Changed 14 months ago by Rahly

the 0.8.4 patch works with the unstable 0.8.5 code. I couldn't get it to work with the latest svn, but then again, i couldn't even compile it without the patch. When the svn is back to working, it should work with it, that i've seen.

  Changed 14 months ago by Rahly

Works with the latest SVN, must have been fixed.

  Changed 13 months ago by nao.nakashima@gmail.com

With 0.8.4 patch only bolds active seeding torrents, don`t see any color.

  Changed 11 months ago by Rahly

you need to set the colors in the configuration. please read the messages. Default is to use current colors.

my configuration options are as followed:

done_fg_color = 2
active_fg_color = 1

  Changed 11 months ago by Rahly

make sure you put that in your ~/.rtorrent.rc

Changed 8 months ago by sleip

updated rahlys patch to 0.8.6

  Changed 8 months ago by Rahly

Wow, nice, no code changes, but it looks like he moved quite a bit of functions around so the patcher failed. Lovely ;-)

follow-up: ↓ 23   Changed 6 months ago by alexei

@Rahly, patcher fails in what version? For me, patch worked correctly for 0.8.6 source. Nice patch, by the way!

in reply to: ↑ 22   Changed 6 months ago by anonymous

Replying to alexei:

@Rahly, patcher fails in what version? For me, patch worked correctly for 0.8.6 source. Nice patch, by the way!

Oooops, never mind. I see what you mean. Sorry about stupid questions.

  Changed 6 months ago by calico@noichi.net

I tried latest patch for 0.86 on changeset 1145, and I can't compile rtorrent. Here is the output error :

make[3]: Entering directory `/root/svn/rtorrent/src/display' g++ -DHAVE_CONFIG_H -I. -I../.. -I. -I./.. -I../.. -g -O2 -g -DDEBUG -I/usr/include/sigc++-2.0 -I/usr/lib/sigc++-2.0/include -I/usr/local/include -I/usr/local/include -I/usr/local/include -MT window_download_list.o -MD -MP -MF .deps/window_download_list.Tpo -c -o window_download_list.o window_download_list.cc window_download_list.cc: In member function 'virtual void display::WindowDownloadList::redraw()': window_download_list.cc:104: error: 'class torrent::Download' has no member named 'up_rate' window_download_list.cc:109: error: 'class torrent::Download' has no member named 'is_active' window_download_list.cc:110: error: 'class torrent::Download' has no member named 'down_rate' make[3]: *** [window_download_list.o] Error 1 make[3]: Leaving directory `/root/svn/rtorrent/src/display'

Someone have an idea ?

Thanks a lot.

Changed 5 months ago by hyper_ch

Canvas Color Patch designed for rev. 1163

  Changed 5 months ago by hyper_ch

of course many thanks go to the original author and to rakshasa's patience with a couple of problems I had adjusting it to rev. 1163

  Changed 4 months ago by anonymous

would be kind of nice to have to added to the main branch, so we wouldn't have to keep rebuilding the patch

  Changed 2 months ago by Carol <caroll@yahoo.com>

This bug should be fixed through  writing services.

  Changed 2 months ago by anonymous

Nice to share your experience. And the bug seems to be in good use.  http://www.astrabeds.com/latex-mattresses.html

  Changed 2 months ago by placement argent

Thanks for taking the time to discuss this, I feel strongly about information and love learning more on this. If possible, as you gain expertise, It is extremely helpful for me. would you mind updating your blog with more information?  http://placementargent.org

  Changed 8 weeks ago by betclic

  Changed 6 weeks ago by anonymous

Thanks for the fine tips and posting..

  Changed 5 weeks ago by nfl jerseys

Alisan Ghase is a retired NFL player who now owns a NFL Jerseys website:  http://www.stajump.com

  Changed 4 weeks ago by anonymous

of course many thanks go to the original author and to rakshasa's patience with a couple of problems I had adjusting it to rev. 1163  http://www.highdesign.ro/

  Changed 3 weeks ago by xbox live cards

I came to your article from another article and am really interested in this learning about this. , I feel strongly about information and love learning more on this. If possible, as you gain expertise, It is extremely helpful for me.  http://www.pcgamesupply.com/buy/Xbox-Live-3-Month-Gold-Card-Code

  Changed 2 weeks ago by hammock chair stand

She seems to be missing in these days. I never heard of her recently. Don't tell me she's quit.  http://www.esleepmasters.com/Hammock_Chair_Stands_s/1869.htm

  Changed 5 days ago by robin

This article gives the light in which we can observe the reality. This is very nice one and gives in depth information. Thanks for this nice article. Good post.....Valuable information for all.  carpet cleaner Monterey

Add/Change #1382 (Colors and Bug Fix)

Author


E-mail address and user name can be saved in the Preferences.


Action
as new
 
Note: See TracTickets for help on using tickets.