This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 227793 - [tml][vncviewer][patch] Implementation of the several encodings, performance enhancement etc
Summary: [tml][vncviewer][patch] Implementation of the several encodings, performance ...
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Sequoyah (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Daniel Barboza Franco CLA
QA Contact:
URL:
Whiteboard: released_0.1
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-04-18 11:53 EDT by Eugene Melekhov CLA
Modified: 2016-11-08 14:45 EST (History)
3 users (show)

See Also:


Attachments
Snapshot of the sources (183.56 KB, application/octet-stream)
2008-04-18 11:53 EDT, Eugene Melekhov CLA
no flags Details
Snapshot of the sources (181.35 KB, application/octet-stream)
2008-04-22 11:09 EDT, Eugene Melekhov CLA
fabio.fantato: iplog+
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Eugene Melekhov CLA 2008-04-18 11:53:00 EDT
Created attachment 96614 [details]
Snapshot of the sources

This is the sequel of my work mentioned in the bug 225955.

I've implemented using of the following RFB encodings:
RRE, Hextile, ZRLE, zlib.

Also I've slightly modified the architecture to allow relatively simple implementation of the new encoding algorithms and UI representation elements.
For example there is a new implementation of the Eclipse VNC view (SWTRemoteDisplayImg) which is faster then previous, but doesn't support zoom.

Any comments and questions are welcome. Please feel free to ask any questions about the architecture changes and implementation details.

It's not easy to make the patch from my changes because of the structural changes (new directories, new/deleted/moved packages/files etc), so I'm attaching here the "snapshot" of the sources that I have at the moment. Please note that this snapshot also includes changes made in the patch for bug 225955.
Comment 1 Eugene Melekhov CLA 2008-04-22 11:09:23 EDT
Created attachment 97029 [details]
Snapshot of the sources

Latest version of the sources with replacement for the problematic DesCiper class.
Comment 2 Eugene Melekhov CLA 2008-04-22 11:13:05 EDT
As I've mentioned in the bug 225955 I wrote the replacement for the DesCipher class, so I'm attaching the new version of the sources here.
Comment 3 Eugene Melekhov CLA 2008-04-23 03:31:03 EDT
Just additional note to clarify the licensing issues:

I wrote the code of all new classes that contained in the archive. I has the right to contribute the code to Eclipese. New source files do not contain the EPL License header, but I do not object if the EPL License header will be added to all files that I've contributed.
Comment 4 Fabio Fantato CLA 2008-04-29 15:44:43 EDT
The ipzilla opened from the bug#225955 was redirected for this bug. 
IPZilla: https://dev.eclipse.org/ipzilla/show_bug.cgi?id=2261
Comment 5 Fabio Fantato CLA 2008-06-02 13:39:06 EDT
This code was verified by TmL committers and it was prepared to be upload. The PMC votting was already started we already had the enough votes to proceed with this contribution. We will sumarize these votes and attach them to IPZilla. 
Comment 6 Fabio Fantato CLA 2008-06-04 14:19:58 EDT
Daniel,
The CQ for this bug was approved in Parallel IP and it can be checkinto CVS.
Please do it and verify if this patch was integrated succesfully.

Eugene,
We was a quite slow in this process but finally we are be able to upload your contribution in our repository. Thanks again for this contribution. It will be really important to make TmL an successful project.

Thanks 
Fabio
Comment 7 Daniel Barboza Franco CLA 2008-07-15 14:59:22 EDT
I finally finished the integration and merge of Eugene's branch. I had to make some adapts due to the parallel branch (the new protocol mechanism).
I also made a little improvement into the pixel reading methods.

Tks,
Daniel.
Comment 8 Fabio Fantato CLA 2008-11-13 08:18:15 EST
CVS branch for 0.1 release: branch_dsdp_tml_official_0_1_0_R