Placed ampersands and asterisks against type name

[crozhon]

Placed ampersands and asterisks against type name as described in the style guide. Also placed asterisks against the type name and removed the space for casts.

Ex. (int*) instead of (int *)

[archshift]

LGTM

[lioncash]

You missed a few pointers (around where you change the wxStaticBitmap
pointers).

I'd comment directly on the line but all Android apps for Github are awful
and don't allow this.
On Aug 13, 2014 5:48 PM, "Charles Rozhon" notifications@github.com wrote:

Placed ampersands and asterisks against type name as described in the
style guide. Also placed asterisks against the type name and removed the
space for casts.

Ex. (int*) instead of (int *)

You can merge this Pull Request by running

git pull https://github.com/ChuckRozhon/dolphin correct_pointer_style_tasinputdlg

Or view, comment on, or merge it at:

#795
Commit Summary

• Placed ampersands and asterisks against type name

File Changes

• M Source/Core/DolphinWX/TASInputDlg.cpp
  https://github.com/dolphin-emu/dolphin/pull/795/files#diff-0 (28)
• M Source/Core/DolphinWX/TASInputDlg.h
  https://github.com/dolphin-emu/dolphin/pull/795/files#diff-1 (6)

Patch Links:

• https://github.com/dolphin-emu/dolphin/pull/795.patch
• https://github.com/dolphin-emu/dolphin/pull/795.diff

—
Reply to this email directly or view it on GitHub
#795.

[crozhon]

Are you referring to wxTextCtrl *textX, *textY? I didn't change these or any others that were multiple definitions on the same line. It wasn't referenced in the style guide at all, so I just went for consistency.

[lioncash]

Oh OK. No worries then.
On Aug 13, 2014 6:59 PM, "Charles Rozhon" notifications@github.com wrote:

Are you referring to wxTextCtrl *textX, *textY? I didn't change these or
any others that were multiple definitions on the same line. It wasn't
referenced in the style guide at all, so I just went for consistency.

—
Reply to this email directly or view it on GitHub
#795 (comment).