Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TIMOB-7363][TIMOB-7425][TIMOB-7455] [1_7_X] barImage handling #1354

Closed
wants to merge 6 commits into from
Closed

[TIMOB-7363][TIMOB-7425][TIMOB-7455] [1_7_X] barImage handling #1354

wants to merge 6 commits into from

Conversation

sptramer
Copy link
Contributor

@sptramer sptramer commented Feb 6, 2012

Duplicate of PR #1252 for 1_7_X.

Should not be reviewed by myself or Vishal. But see comments below regarding the other cherry-picks to resolve the regressions introduced by the original fix. All three tickets should be tested as well as a KS pass on barImage to ensure that there is no behavior difference from vanilla 1.7.x.

@vishalduggal
Copy link
Contributor

PR #1252 introduced a regression which is filed in TIMOB-7425. This was fixed by PR #1280.

PR #1280 introduced a regression which was filed in TIMOB-7455. This was fixed by PR #1292.

Max Stepanov and others added 2 commits February 6, 2012 12:54
@srahim
Copy link
Contributor

srahim commented Feb 14, 2012

Should this PR need commits from PR #1292 ?

@srahim
Copy link
Contributor

srahim commented Feb 14, 2012

Missing two more commits
From
PR #1280 or #1278 commit 7c0288d

and
for PR #1292 commit 54f1926

is missing...

@srahim
Copy link
Contributor

srahim commented Feb 14, 2012

Code Reviewed ... as vishal noted barimage is not being allocated... i think there was merge error while cherry picking.

@sptramer
Copy link
Contributor Author

This comment needs to be more specific. I'm not seeing where in the code this is missing based on diffs.

@sptramer
Copy link
Contributor Author

Closing this PR to submit a new one based on diff patching.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants