Add parameter to include source code in the disassembly

[awujek]

Signed-off-by: Adam Wujek [email protected]

[awujek]

can be considered as related to #21

[HBehrens]

Thank you, @awujek !

Do you think the default should be True instead of the picked False?

Also, could you do me a favour and add a test similar to

puncover/tests/test_collector.py

Lines 62 to 77 in 706dcc5

	def test_parses_assembly2(self):
	assembly = """
	00000098 <pbl_table_addr.constprop.0>:
	pbl_table_addr():
	98: a8a8a8a8 .word 0xa8a8a8a8
	0000009c <__aeabi_dmul>:
	__aeabi_dmul():
	9c: b570 push {r4, r5, r6, lr}
	"""
	c = Collector(None)
	self.assertEqual(2, c.parse_assembly_text(assembly))
	self.assertTrue(0x0000009c in c.symbols)
	self.assertEqual(c.symbols[0x0000009c]["name"], "__aeabi_dmul")
	self.assertTrue(0x00000098 in c.symbols)
	self.assertEqual(c.symbols[0x00000098]["name"], "pbl_table_addr")

that shows that the parser does indeed skip over source code lines?

[awujek]

Thank you, @awujek !

You're welcome!

Do you think the default should be True instead of the picked False?

It is up to you. I wanted to keep it backward compatible;) But as a new user I would prefer to have the source by default:)

Also, could you do me a favour and add a test similar to

puncover/tests/test_collector.py

Lines 62 to 77 in 706dcc5

	def test_parses_assembly2(self):
	assembly = """
	00000098 <pbl_table_addr.constprop.0>:
	pbl_table_addr():
	98: a8a8a8a8 .word 0xa8a8a8a8
	0000009c <__aeabi_dmul>:
	__aeabi_dmul():
	9c: b570 push {r4, r5, r6, lr}
	"""
	c = Collector(None)
	self.assertEqual(2, c.parse_assembly_text(assembly))
	self.assertTrue(0x0000009c in c.symbols)
	self.assertEqual(c.symbols[0x0000009c]["name"], "__aeabi_dmul")
	self.assertTrue(0x00000098 in c.symbols)
	self.assertEqual(c.symbols[0x00000098]["name"], "pbl_table_addr")

that shows that the parser does indeed skip over source code lines?

ok, let me check

[HBehrens]

Sorry for the delay, @awujek – I hope that I will be able to get to this over the weekend.