Re: [VOTE SUMMARY] Mehdi Houshmand for FOP committer

2011-11-17 Thread Chris Bowditch

On 16/11/2011 09:59, mehdi houshmand wrote:

Hi Chris,


Hi Mehdi,



mehdi isn't taken, so if you could set me up with that ID.


Sure, we will request Mehdi as your ID


Thank you all for your positive messages, I'll endeavour to ensure FOP
remains the high quality product you have developed.


Whilst you are waiting for your account to be setup, I suggest you have 
a read through the following page:


http://apache.org/dev/committers.html



Mehdi

Thanks,

Chris




On 16 November 2011 09:35, Chris Bowditchbowditch_ch...@hotmail.com  wrote:

On 10/11/2011 14:18, Chris Bowditch wrote:

Dear FOP committers,

The vote has been running for 6 days now so it's time to sum up the votes.

Mehdi has submitted dozens of high quality patches and regularly
participates on the mailing lists. Mehdi is already an official FOP
Contributor. I propose Mehdi Houshmand as a FOP committer.

I vote +1

There were 6 +1s, no 0's or vetos, so the vote passes.

Congratulations Mehdi on becoming a committer. Please send me your preferred
login ID so I can create your account. The following page will help with the
task of making sure you pick a unique ID:

http://people.apache.org/committer-index.html

Thanks,

Chris


Replies to general@ please.

Thanks,

Chris










DO NOT REPLY [Bug 52192] [PATCH] TTFFontLoader ignores the useKerning attribute used with fonts

2011-11-17 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52192

--- Comment #1 from Chris Bowditch bowditch_ch...@hotmail.com 2011-11-17 
11:20:50 UTC ---
Hi Mehdi,

Your test case contains a bug. You try to build the URL manually:

   File url = new File(test/resources/fonts/DejaVuLGCSerif.ttf);
String absoluteFilePath = file:// + url.getAbsolutePath();

This fails to run on Windows with the following error:

   [junit] java.net.UnknownHostException: C
   [junit] at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:177)
   [junit] at java.net.Socket.connect(Socket.java:524)
   [junit] at java.net.Socket.connect(Socket.java:474)

Why not just get the String version of the URL instead of trying to build the
URL through String concatenation?

Thanks,

Chris

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 52192] [PATCH] TTFFontLoader ignores the useKerning attribute used with fonts

2011-11-17 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52192

Mehdi Houshmand med1...@gmail.com changed:

   What|Removed |Added

  Attachment #27947|0   |1
is obsolete||

--- Comment #2 from Mehdi Houshmand med1...@gmail.com 2011-11-17 11:25:35 UTC 
---
Created attachment 27962
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=27962
changed URL retrieval mechanism

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 52192] [PATCH] TTFFontLoader ignores the useKerning attribute used with fonts

2011-11-17 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52192

Chris Bowditch bowditch_ch...@hotmail.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #3 from Chris Bowditch bowditch_ch...@hotmail.com 2011-11-17 
12:11:07 UTC ---
Thanks Mehdi. Patch applied in revision 1203163

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 52197] [PATCH] AFM encoding

2011-11-17 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52197

Mehdi Houshmand med1...@gmail.com changed:

   What|Removed |Added

  Attachment #27953|0   |1
is obsolete||

--- Comment #3 from Mehdi Houshmand med1...@gmail.com 2011-11-17 16:07:15 UTC 
---
Created attachment 27963
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=27963
patch amendment

This patch replaced the old patch, the TTFFontLoaderTestCase uses font in:
test/resources/fonts/DejaVuLGCSerif.ttf

This is amended to following commit#r1203163
test/resources/fonts/ttf/DejaVuLGCSerif.ttf

Same file moves apply:
test/resources/fonts/{,ttf/}glb12.README.txt
test/resources/fonts/{,ttf/}glb12.ttf.xml
test/resources/fonts/{,ttf/}glb12.ttf
test/resources/fonts/{,ttf/}DejaVuLGCSerif.ttf
test/resources/fonts/{,ttf/}glb12.ttf.ansi.xml
test/resources/fonts/{,ttf/}DejaVuLGCSerif.LICENSE
test/events/font.fo - test/java/org/apache/fop/fonts/substituted-font.fo

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 52206] New: [PATCH] Small performance improvements by adding static declarations

2011-11-17 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52206

 Bug #: 52206
   Summary: [PATCH] Small performance improvements by adding
static declarations
   Product: Fop
   Version: 1.1dev
  Platform: All
OS/Version: All
Status: NEW
  Severity: normal
  Priority: P2
 Component: fo tree
AssignedTo: fop-dev@xmlgraphics.apache.org
ReportedBy: alex.gio...@gmail.com
Classification: Unclassified


Created attachment 27967
  -- https://issues.apache.org/bugzilla/attachment.cgi?id=27967
Patch adding static declarations and removing 2 trace messages

I was profiling an application running FOP 1.0 and found a few hotspots. Some
of them have been already fixed in trunk ( e.g. Bug 50626 ). 

The ones requiring minimal changes are contained in this patch. It adds the
'static' declaration for loggers and removes 2 heavily used trace logging
messages. I avoided to include bigger changes (mainly in FObj which would
benefit by using a LinkedList instead of the siblings array) as bigger patches
seem to be difficult to process ( #46962 )

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.