[Bug 69935] ClientAnchor: Inconsistent types between getCol1()/getCol2() returning short and setCol1()/setCol2() accepting int

2026-02-14 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69935

PJ Fanning  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from PJ Fanning  ---
PR merged

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



[Bug 69935] ClientAnchor: Inconsistent types between getCol1()/getCol2() returning short and setCol1()/setCol2() accepting int

2026-02-08 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69935

--- Comment #3 from [email protected] ---
I’ve opened a PR for this issue:

https://github.com/apache/poi/pull/1006

The change adds default short-based setters `setCol1(short)` and
`setCol2(short)` on `ClientAnchor` that delegate to the existing int-based
setters, and includes a small regression test (`TestClientAnchorShortSetter`)
to verify both the new and existing setters. 
The `@since` tag is left as `TBD` for the committer to adjust to the
appropriate release version.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



[Bug 69935] ClientAnchor: Inconsistent types between getCol1()/getCol2() returning short and setCol1()/setCol2() accepting int

2026-01-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69935

--- Comment #2 from [email protected] ---
a PR to ADD missing versions of the setters so they match, e.g. setCol1( short
) could do the trick.  java supports method overloading.  

would merely adding the missing setters for short satisfy kotlin support? 
probably need to make sure any reflective code in POI doesn't get confused by
multiple method redefinitions on the affected objects

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



[Bug 69935] ClientAnchor: Inconsistent types between getCol1()/getCol2() returning short and setCol1()/setCol2() accepting int

2026-01-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69935

--- Comment #1 from PJ Fanning  ---
Unfortunately making breaking changes causes more unhappy users than changing
old APIs to suit a very small number of people.

If this is a big issue for you, you can write your own lib method that wraps
the call that you don't like.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



[Bug 69935] ClientAnchor: Inconsistent types between getCol1()/getCol2() returning short and setCol1()/setCol2() accepting int

2026-01-22 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=69935

Matafagafo  changed:

   What|Removed |Added

 CC||[email protected]

-- 
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]