Can I get rid of these?

I'm refactoring someone else's code and I keep coming across instances of these:

Code:

---

someClass** sc;
*sc = pointer to something somewhere else;
... (*((*sc))).someMember; //note the double parentheses

---

The code as it is compiles and works, but the double parentheses are an eyesore, and I'd like to clean them up as:

Code:

---

(*sc)->someMember;

---

Would I be breaking anything here by doing this?

Quote:

---

Would I be breaking anything here by doing this?

---

No, they are equivalent.

I don't think you need the double ones.

Would there be any reason why this person would have coded like this in the first place? Personal quirk?

Style. Fun. Looks cool. Looks better. In someone's point of view, at least. Pick one. There's no advantage in the language in doing so.

Quote:

---

Originally Posted by cunnus88

The code as it is compiles and works, but the double parentheses are an eyesore, and I'd like to clean them up as:

Code:

---

(*sc)->someMember;

---

---

That's fine, and looks way better. If you're doing a bunch of accesses through (*sc), you can make a reference to alias it which cleans things up even further. Instead of the above, you could:

Code:

---

someClass &scref = **sc;

scref.someMember;

---

I tend to do

Code:

---

(**sc).someMember;

---