Potential Bug in mpd_qresize and mpd_qresize_zero

[KenCheung]

BPO	15059
Nosy	@skrah
Files	mpdecimal_clone 2.txt

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2012-06-14.07:13:01.641>
created_at = <Date 2012-06-13.17:52:37.319>
labels = ['extension-modules', 'type-bug', 'invalid']
title = 'Potential Bug in mpd_qresize and mpd_qresize_zero'
updated_at = <Date 2012-06-14.07:13:01.640>
user = 'https://bugs.python.org/KenCheung'

bugs.python.org fields:

activity = <Date 2012-06-14.07:13:01.640>
actor = 'skrah'
assignee = 'none'
closed = True
closed_date = <Date 2012-06-14.07:13:01.641>
closer = 'skrah'
components = ['Extension Modules']
creation = <Date 2012-06-13.17:52:37.319>
creator = 'Ken.Cheung'
dependencies = []
files = ['25951']
hgrepos = []
issue_num = 15059
keywords = []
message_count = 2.0
messages = ['162719', '162756']
nosy_count = 2.0
nosy_names = ['skrah', 'Ken.Cheung']
pr_nums = []
priority = 'normal'
resolution = 'not a bug'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue15059'
versions = ['Python 3.3']

[KenCheung]

I observed a code clone from the following files.

function : mpd_qresize @ (file: "Python-3.3.0a2/Modules/_decimal/libmpdec/mpdecimal.c", line: 481)~493
function : mpd_qresize_zero @ (file: "Python-3.3.0a2/Modules/_decimal/libmpdec/mpdecimal.c", line: 501)~517

The two functions perform different in the else if condition. I wonder this is due to functionality requirement or a bug. The source code is included in the attachment. Hope it helps.

[skrah]

You can't return mpd_realloc_dyn(result, nwords, status) in the
second instance: the coefficient must be initialized to zero
later on. So this is intentional.