Do you own a Debenu Quick PDF Library version 7, 8, 9, 10, 11, 12, 13 or iSEDQuickPDF license? Upgrade to Debenu Quick PDF Library 14 today!

Debenu Quick PDF Library - PDF SDK Community Forum Homepage
Forum Home Forum Home > For Users of the Library > General Discussion
  New Posts New Posts RSS Feed - QuickPDF v5.15 Release Candidate 1
  FAQ FAQ  Forum Search   Register Register  Login Login

QuickPDF v5.15 Release Candidate 1

 Post Reply Post Reply
Author
Message
Michel_K17 View Drop Down
Newbie
Newbie
Avatar
www.exp-systems.com

Joined: 25 Jan 03
Status: Offline
Points: 297
Post Options Post Options   Thanks (0) Thanks(0)   Quote Michel_K17 Quote  Post ReplyReply Direct Link To This Post Topic: QuickPDF v5.15 Release Candidate 1
    Posted: 18 Oct 06 at 12:36AM

   Just a quick posts for the programmers and testers who have access to the full source code download section of the forum, we now have a new v5.15 Release Candidate 1 provided courtesy of Marian Pascalau. We are still finalizing the list of changes and documentation, but in the mean time, you can find out more details and download the new library [here].

   As testing progresses, we will make the compiled library available to everybody soon.

Michel
Back to Top
hbarclay View Drop Down
Team Player
Team Player


Joined: 29 Oct 05
Location: United States
Status: Offline
Points: 39
Post Options Post Options   Thanks (0) Thanks(0)   Quote hbarclay Quote  Post ReplyReply Direct Link To This Post Posted: 18 Oct 06 at 9:50AM
I have a real problem with this release. I have just begun to evaluate it, so I don't really know how it is going to function yet, but it appears there have been a number of useless and unnecessary changes made to the code.

For example look at the function GetFormFieldBound in the unit uiSEDQuickPDF. Someone has reformatted the text to make it look like what their idea of pretty code is. There have been no changes to the code, and yet when it is run through vdiff, it shows to have had every line changed. This is a real problem and makes evaluation of changes much harder than they should be.

PLEASE DON'T CHANGE THE CODE UNLESS IT NEEDS TO BE CHANGED!

Back to Top
Ingo View Drop Down
Moderator Group
Moderator Group
Avatar

Joined: 29 Oct 05
Status: Offline
Points: 3524
Post Options Post Options   Thanks (0) Thanks(0)   Quote Ingo Quote  Post ReplyReply Direct Link To This Post Posted: 18 Oct 06 at 10:28AM
Hi Harry!

Marian had made the code more readable (for himself).
To evaluate you should take the freeware
WinDiff. There you should activate the option "Ignore White Space" - Then it's more easier.

Best regards,
Ingo
Back to Top
hbarclay View Drop Down
Team Player
Team Player


Joined: 29 Oct 05
Location: United States
Status: Offline
Points: 39
Post Options Post Options   Thanks (0) Thanks(0)   Quote hbarclay Quote  Post ReplyReply Direct Link To This Post Posted: 18 Oct 06 at 10:44AM
Originally posted by Ingo Ingo wrote:

Hi Harry!

Marian had made the code more readable (for himself).
To evaluate you should take the freeware
WinDiff. There you should activate the option "Ignore White Space" - Then it's more easier.

Best regards,
Ingo


I understand the reason, I just think it is a major mistake. No change should be made for any reason other than to fix a problem, or to add a feature. What is easier for one person is harder for someone else. Unnecessary changes are just asking for problems.

Ignoring white space will not solve the problem. In many cases declarations were moved from one line to another and show up as changes. The code has been altered in ways that were just completely uncalled for and should never happen.

PLEASE DON'T CHANGE THE CODE UNLESS IT NEEDS TO BE CHANGED!
Back to Top
swb1 View Drop Down
Debenu Quick PDF Library Expert
Debenu Quick PDF Library Expert
Avatar

Joined: 05 Dec 05
Location: United States
Status: Offline
Points: 100
Post Options Post Options   Thanks (0) Thanks(0)   Quote swb1 Quote  Post ReplyReply Direct Link To This Post Posted: 18 Oct 06 at 11:12AM

I agree with Harry.

 

There is no need to submit code changes where there is in fact no change. I understand the need to re-arrange code for readability and I often do. However when submitting final code there should be a consistent format and given that we are many developers working with many styles I humbly recommend that we submit code with the format that our predecessor(s) used.

 

I look forward to evaluating the new source, however it seems that the source forum password has changed since the last time I accessed it. Hopefully Ingo or Michel will send me the new password soon ;) pdfguy[at]dashdev[dot]com

 

BTW: BeyondCompare from http://www.scootersoftware.com/ is the world’s most awesome file comparison tool, particularly for Delphi Code.

Back to Top
hbarclay View Drop Down
Team Player
Team Player


Joined: 29 Oct 05
Location: United States
Status: Offline
Points: 39
Post Options Post Options   Thanks (0) Thanks(0)   Quote hbarclay Quote  Post ReplyReply Direct Link To This Post Posted: 18 Oct 06 at 3:47PM
swb1,

Your reply was much more diplomatic than mine and that's probably the tone I should have taken with my original post. I really don't intend to belittle all the work that has gone into making these changes, but I think this is very important. I have spent some time looking through the code.
We use Starteam for our source control and I normally just use the vdiff that comes with Starteam, but for a comparison of entire directories I use Ultracompare (http://www.ultraedit.com/). Either product makes it very simple to do quality control checks on changes. Unfortunately with the source code in the 5.15 Release Candidate 1 there are so many cosmetic changes that it becomes practically useless.

It is critical to the quality of the code base not to make unnecessary changes.

Thanks
Harry
Back to Top
Michel_K17 View Drop Down
Newbie
Newbie
Avatar
www.exp-systems.com

Joined: 25 Jan 03
Status: Offline
Points: 297
Post Options Post Options   Thanks (0) Thanks(0)   Quote Michel_K17 Quote  Post ReplyReply Direct Link To This Post Posted: 19 Oct 06 at 12:48AM
   I will let Marian respond to the criticism. I agree that diplomacy is key since we are all volunteers. Marian is a very nice fellow and some of the work that I have seen him do in the past was top notch. What I am really interested in: what were the changes and bug fixes: hopefully Marian can provide us with a list soon.

By the way, to compile the code successfully, you may need to create a DCU and an EXE subfolder to ensure that the project compiles properly. Frankly, I like the idea of organizing the output files.

Anyway, let's give this new version a fair shake. In 4 weeks, we'll summarize where we are at, and what to do next.
Back to Top
Ingo View Drop Down
Moderator Group
Moderator Group
Avatar

Joined: 29 Oct 05
Status: Offline
Points: 3524
Post Options Post Options   Thanks (0) Thanks(0)   Quote Ingo Quote  Post ReplyReply Direct Link To This Post Posted: 19 Oct 06 at 2:08AM
Hi!

I've seen that few changes posted in section
"Submissions for changes for v5.15"
(specially few from JanN) are missed in the sources.

To all source-testers:
Please check which posted modifications from "Submissions for changes for v5.15" should be implemented, too and post the links here. So we (i?) can insert it in the second beta or the final release.

Thanks again to Marian for his efforts! Is there anybody else wanted to dig more deeper in the code? I can't imagine that Marian is the only one?!

Best regards,
Ingo



Back to Top
swb1 View Drop Down
Debenu Quick PDF Library Expert
Debenu Quick PDF Library Expert
Avatar

Joined: 05 Dec 05
Location: United States
Status: Offline
Points: 100
Post Options Post Options   Thanks (0) Thanks(0)   Quote swb1 Quote  Post ReplyReply Direct Link To This Post Posted: 19 Oct 06 at 11:29AM

Kudos to Marian - very nice work! You obviously spent a great deal of time on this and I would not want to see your efforts on this release to go under-appreciated.

 

I wish that I had more time to devote to this; unfortunately, my circumstance only allows me to work on issues that directly affect my projects.

 

Thanks Again

 

Steve

Back to Top
marian_pascalau View Drop Down
Debenu Quick PDF Library Expert
Debenu Quick PDF Library Expert


Joined: 28 Mar 06
Location: Germany
Status: Offline
Points: 278
Post Options Post Options   Thanks (0) Thanks(0)   Quote marian_pascalau Quote  Post ReplyReply Direct Link To This Post Posted: 26 Oct 06 at 9:18AM

Hi, great to see that someone is checking my results. I am not happy to see when someone is checking unnecessary my cosmetic work(expected to see more checks on program results) but I have to admit that I am happy to see some feedback at all.

First I see that everybody wants to know why I had to change the source code layout. The reason is that I use QPDF.ActiveX in one of my projects and because I hat a lot of troubles with rendering I changed and checked everything what from my point of view was suspicious. The cosmetic change was necessary because I found most of the times small troubles hidden inside the old compacted layout and I wanted to understand the whole logic before I go through with a new design.

This way I was able not only to identify logic problems but also points of interest for a future redesign.

I understand that someone may hope that the development will continue from the old base code, but as long as the manufacturer is not active I believe is better to go further with a new design.

I understand that this change may scare you but without this change there is no hope for a real next release and my work is finished. At the end we don't want to keep the old not-yet-detected stuff but fix them and continue with that what is missing (at least that was my intention when I joined this community).

For your information all of my changes regards exclusive the rendering capabilities. Everything else comes from discussions with other members. Most of my changes are commented inside the source code (search for "Marian" or "// <"Marian" ) and you will see:

  • enhancements (annotation processing (document link, Url, external link))
  • logic improvements (like added "else" between "if"'s)
  • remove redundant blocks
  • rewrite entire logic: in image processing I process data without buffering + correction on processing image operation (inversed image or image with mask).

... to be continued.



Edited by marian_pascalau
Back to Top
marian_pascalau View Drop Down
Debenu Quick PDF Library Expert
Debenu Quick PDF Library Expert


Joined: 28 Mar 06
Location: Germany
Status: Offline
Points: 278
Post Options Post Options   Thanks (0) Thanks(0)   Quote marian_pascalau Quote  Post ReplyReply Direct Link To This Post Posted: 26 Oct 06 at 9:52AM

Hi, before I continue I have a question for Ingo.

Ingo, you said that some changes are missing from sources. Can you please be more specific about which changes you are talking about?

Back to Top
marian_pascalau View Drop Down
Debenu Quick PDF Library Expert
Debenu Quick PDF Library Expert


Joined: 28 Mar 06
Location: Germany
Status: Offline
Points: 278
Post Options Post Options   Thanks (0) Thanks(0)   Quote marian_pascalau Quote  Post ReplyReply Direct Link To This Post Posted: 26 Oct 06 at 10:58AM

.... before I will not forget I want to thank everybody for your feedback. I have spent a lot of time improving on this library and I'm sure that with the propper support we can go further.

Regards, Marian

Back to Top
Ingo View Drop Down
Moderator Group
Moderator Group
Avatar

Joined: 29 Oct 05
Status: Offline
Points: 3524
Post Options Post Options   Thanks (0) Thanks(0)   Quote Ingo Quote  Post ReplyReply Direct Link To This Post Posted: 26 Oct 06 at 6:10PM
Hi Marian!

I mean the two things from Uli and Jan at
http://www.quickpdf.org/forum/forum_topics.asp?FID=17

The modifications i've sent to you are from forum members via personal email but here are modifications, too. So i've thought perhaps we'll forget a bit ;-)

Best regards,
Ingo
Back to Top
marian_pascalau View Drop Down
Debenu Quick PDF Library Expert
Debenu Quick PDF Library Expert


Joined: 28 Mar 06
Location: Germany
Status: Offline
Points: 278
Post Options Post Options   Thanks (0) Thanks(0)   Quote marian_pascalau Quote  Post ReplyReply Direct Link To This Post Posted: 26 Oct 06 at 6:39PM

Ingo, I did not forget about those.

The change proposed by Ulrich was already implemented. Because of that there is no information about Ulrich in that place. Anyway if you check the sourcecode you will see major changes around that place (code optimization + other bug corrections).

The second suggestion proposed by IanN is a little bit complicated and I was not able to confirm the reality of this problem. I have profiled a lot of times my sourcecode and I was never able to identify performace penalty caused by PDF page selection. The only penalty I may confirm is the processing of PDF Images. Because of that I skiped over this change till some real proof (email with PDF sample to me) or time is available.

Regards, Marian

Back to Top
ixm7 View Drop Down
Senior Member
Senior Member


Joined: 13 Jan 06
Status: Offline
Points: 68
Post Options Post Options   Thanks (0) Thanks(0)   Quote ixm7 Quote  Post ReplyReply Direct Link To This Post Posted: 29 Oct 06 at 9:51AM
Marian,

Does the new version solve the problem with processing large pdf files? See discussion at: http://www.quickpdf.org/forum/forum_posts.asp?TID=278&PN=1

Many Thanks!
- Ido
Back to Top
marian_pascalau View Drop Down
Debenu Quick PDF Library Expert
Debenu Quick PDF Library Expert


Joined: 28 Mar 06
Location: Germany
Status: Offline
Points: 278
Post Options Post Options   Thanks (0) Thanks(0)   Quote marian_pascalau Quote  Post ReplyReply Direct Link To This Post Posted: 30 Oct 06 at 7:49AM

Hi Ido,

Because I cannot confirm that you have the same problem it is impossible to give you a categoric answer. Would you be so kind and give it yurself a try with the 5.15 version? Of course that I can do a test myself, but as I say always :-) please send me a sample PDF + sample Delphi/C++/VB/c#/VB.NET snippet to test your problem.

Regards, Marian

Back to Top
ixm7 View Drop Down
Senior Member
Senior Member


Joined: 13 Jan 06
Status: Offline
Points: 68
Post Options Post Options   Thanks (0) Thanks(0)   Quote ixm7 Quote  Post ReplyReply Direct Link To This Post Posted: 30 Oct 06 at 8:01AM
Hi Marian,

The speed/hanging problem I've experienced occurred even with pdf files with no images. It seems to be just a function of number of pages.

I think Jan's code change is probably the needed fix and I hope it makes its way into this release.

Cheers,
- Ido
Back to Top
marian_pascalau View Drop Down
Debenu Quick PDF Library Expert
Debenu Quick PDF Library Expert


Joined: 28 Mar 06
Location: Germany
Status: Offline
Points: 278
Post Options Post Options   Thanks (0) Thanks(0)   Quote marian_pascalau Quote  Post ReplyReply Direct Link To This Post Posted: 20 Nov 06 at 12:00PM

Hi Ido,

FYI, The next version of QuickPDF will solve that speed problem.

Back to Top
ixm7 View Drop Down
Senior Member
Senior Member


Joined: 13 Jan 06
Status: Offline
Points: 68
Post Options Post Options   Thanks (0) Thanks(0)   Quote ixm7 Quote  Post ReplyReply Direct Link To This Post Posted: 20 Nov 06 at 12:06PM
Thanks!!!
- Ido
Back to Top
 Post Reply Post Reply
  Share Topic   

Forum Jump Forum Permissions View Drop Down

Forum Software by Web Wiz Forums® version 11.01
Copyright ©2001-2014 Web Wiz Ltd.

Copyright © 2017 Debenu. Debenu Quick PDF Library is a PDF SDK. All rights reserved. About — Contact — Blog — Support — Online Store